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

Pass more complete cstr in Rewrite.transitivity strategy #15577

Merged
merged 1 commit into from Feb 5, 2022

Conversation

SkySkimmer
Copy link
Contributor

Fix #15568 (on v8.15)
Not sure if this fixes anything on master.

@SkySkimmer SkySkimmer added this to the 8.15.1 milestone Jan 31, 2022
@SkySkimmer SkySkimmer requested a review from a team as a code owner January 31, 2022 14:41
@SkySkimmer SkySkimmer added the kind: fix This fixes a bug or incorrect documentation. label Feb 3, 2022
@SkySkimmer SkySkimmer requested review from a team and removed request for a team February 3, 2022 10:19
Fix coq#15568 (on v8.15)
Not sure if this fixes anything on master.
@ppedrot
Copy link
Member

ppedrot commented Feb 3, 2022

Is this targetting master then?

@ppedrot ppedrot self-assigned this Feb 5, 2022
@ppedrot
Copy link
Member

ppedrot commented Feb 5, 2022

@SkySkimmer ping

@SkySkimmer
Copy link
Contributor Author

Is this targetting master then?

yes

@ppedrot
Copy link
Member

ppedrot commented Feb 5, 2022

@coqbot merge now

@coqbot-app coqbot-app bot merged commit 7c608fe into coq:master Feb 5, 2022
@coqbot-app coqbot-app bot added this to Request 8.15.1 inclusion in Coq 8.15 Feb 5, 2022
@SkySkimmer SkySkimmer deleted the transitivity-opt-rew-rel branch February 7, 2022 16:32
SkySkimmer added a commit to SkySkimmer/coq that referenced this pull request Mar 18, 2022
@coqbot-app coqbot-app bot moved this from Request 8.15.1 inclusion to Shipped in 8.15.1 in Coq 8.15 Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: fix This fixes a bug or incorrect documentation.
Projects
No open projects
Coq 8.15
  
Shipped in 8.15.1
Development

Successfully merging this pull request may close these issues.

rewrite_strat regression in 8.15
2 participants