-
Notifications
You must be signed in to change notification settings - Fork 685
8.6.1 CHANGES entries. #856
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You ask for feedback so I find something to say, but that's very minor.
CHANGES
Outdated
- Fix outdated description in RefMan. | ||
- Repairing `Set Rewriting Schemes` | ||
- Fixing #5487 (v8.5 regression on ltac-matching expressions with evars). | ||
- remove unneeded -emacs flag in coq-prog-args in test-suite files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Possibly non relevant for this list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed.
CHANGES
Outdated
- Opaque side effects | ||
- Fix #5132: coq_makefile generates incorrect install goal | ||
- Run non-tactic comands without resilient_command | ||
- [ltac] Move dummy plugin to plugins folder. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same remark.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed.
CHANGES
Outdated
- Fix #5380: Default colors for CoqIDE are actually applied.
- Fix plugin warnings
- Use more neutral wording instead of mentioning CoqIDE
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of the messages mentioned CoqIDE specifically, but those messages also apply to PG/xml or vsCoq. See PR #812.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll remove this change from the log if you don't mind, I don't think it brings much to the user to learn about it.
|
||
- Fix #5380: Default colors for CoqIDE are actually applied. | ||
- Fix plugin warnings | ||
- Use more neutral wording instead of mentioning CoqIDE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of the messages mentioned CoqIDE specifically, but those messages also apply to PG/xml or vsCoq. See PR #812.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll remove this change from the log if you don't mind, I don't think it brings much to the user to learn about it.
bbaa371
to
ba57c32
Compare
(cherry picked from commit beb375a)
Any feedback? Missing entries?