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

[Merged by Bors] - feat: require @[simps!] if simps runs in expensive mode #1885

Closed
wants to merge 11 commits into from

Conversation

fpvandoorn
Copy link
Member

@fpvandoorn fpvandoorn commented Jan 27, 2023

  • This does not change the behavior of simps, just raises a linter error if you run simps in a more expensive mode without writing !.
  • Fixed some incorrect occurrences of to_additive, simps. Will do that systematically in future PR.
  • Fix port of OmegaCompletePartialOrder.ContinuousHom.ofMono a bit

Open in Gitpod

@fpvandoorn fpvandoorn added awaiting-review t-meta Tactics, attributes or user commands awaiting-CI and removed awaiting-CI labels Jan 27, 2023
@semorrison semorrison added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Jan 29, 2023
@semorrison semorrison removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Feb 3, 2023
@semorrison semorrison added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Feb 5, 2023
@urkud
Copy link
Member

urkud commented Feb 5, 2023

I'm going to merge master.

@urkud
Copy link
Member

urkud commented Feb 5, 2023

I merged master, now almost all non-meta changes are about adding ! here and there (all but one you mention in the commit message). I'm not an expert in meta programming but meta changes look good to me.

@urkud urkud removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Feb 5, 2023
@urkud
Copy link
Member

urkud commented Feb 5, 2023

maintainer merge

@fpvandoorn
Copy link
Member Author

I'm treating maintainer merge as bors d+

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Feb 6, 2023
bors bot pushed a commit that referenced this pull request Feb 6, 2023
* This does not change the behavior of `simps`, just raises a linter error if you run `simps` in a more expensive mode without writing `!`.
* Fixed some incorrect occurrences of `to_additive, simps`. Will do that systematically in future PR.
* Fix port of `OmegaCompletePartialOrder.ContinuousHom.ofMono` a bit



Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
@bors
Copy link

bors bot commented Feb 6, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: require @[simps!] if simps runs in expensive mode [Merged by Bors] - feat: require @[simps!] if simps runs in expensive mode Feb 6, 2023
@bors bors bot closed this Feb 6, 2023
@bors bors bot deleted the simpsbang branch February 6, 2023 22:55
jcommelin pushed a commit that referenced this pull request Feb 7, 2023
* This does not change the behavior of `simps`, just raises a linter error if you run `simps` in a more expensive mode without writing `!`.
* Fixed some incorrect occurrences of `to_additive, simps`. Will do that systematically in future PR.
* Fix port of `OmegaCompletePartialOrder.ContinuousHom.ofMono` a bit



Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has been sent to bors. t-meta Tactics, attributes or user commands
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants