Skip to content
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

Adapt to MC#1256 #1279

Merged
merged 2 commits into from
Aug 6, 2024
Merged

Adapt to MC#1256 #1279

merged 2 commits into from
Aug 6, 2024

Conversation

Tragicus
Copy link
Collaborator

@Tragicus Tragicus commented Aug 5, 2024

Motivation for this change

math-comp/math-comp#1256 changes the behavior of rewrite (seemingly because it generalizes some operations). This PR makes the necessary changes to keep compiling.
There was also a notation mismatch.

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers

Reference: How to document

Reminder to reviewers

@Tragicus Tragicus force-pushed the pr1256 branch 2 times, most recently from c9eecc1 to 94b3b7b Compare August 5, 2024 13:43
@Tragicus
Copy link
Collaborator Author

Tragicus commented Aug 6, 2024

I can not see any example of that in the changelog, do we declare changing the level of a notation there?

@affeldt-aist
Copy link
Member

Maybe an entry in the "Changed" section is appropriate. Need this be merged ASAP? (I am just waiting for one last green light before releasing.)

@Tragicus
Copy link
Collaborator Author

Tragicus commented Aug 6, 2024

Making it to this release would of course be best, but it is no big deal if it waits for the next one.

@affeldt-aist affeldt-aist merged commit f4efd39 into math-comp:master Aug 6, 2024
22 checks passed
@Tragicus Tragicus deleted the pr1256 branch August 6, 2024 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants