Skip to content

[Merged by Bors] - refactor: move all register_simp_attrs to 1 file#5681

Closed
urkud wants to merge 12 commits intomasterfrom
forward-port-19223
Closed

[Merged by Bors] - refactor: move all register_simp_attrs to 1 file#5681
urkud wants to merge 12 commits intomasterfrom
forward-port-19223

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented Jul 3, 2023

There are slight differences between mathlib3 and mathlib4 (different set of attributes, different lemmas are in core/std), so I redid the same refactor instead of forward-porting changes.

mathlib3 PR: leanprover-community/mathlib3#19223


Open in Gitpod

@urkud urkud added WIP Work in progress mathlib3-pair This PR is a forward-port of a mathlib3 PR or part of one, either under review or recently merged labels Jul 3, 2023
@urkud urkud added awaiting-review and removed WIP Work in progress labels Jul 3, 2023
@jcommelin
Copy link
Copy Markdown
Member

Thanks 🎉

If CI passes, please remove the label awaiting-CI and merge this yourself, by adding a comment bors r+.

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Jul 3, 2023

✌️ urkud can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Jul 3, 2023
@urkud
Copy link
Copy Markdown
Member Author

urkud commented Jul 3, 2023

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Jul 3, 2023
bors bot pushed a commit that referenced this pull request Jul 3, 2023
There are slight differences between `mathlib3` and `mathlib4` (different set of attributes, different lemmas are in core/std), so I redid the same refactor instead of forward-porting changes.

mathlib3 PR: leanprover-community/mathlib3#19223
@bors
Copy link
Copy Markdown

bors bot commented Jul 3, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title refactor: move all register_simp_attrs to 1 file [Merged by Bors] - refactor: move all register_simp_attrs to 1 file Jul 3, 2023
@bors bors bot closed this Jul 3, 2023
@bors bors bot deleted the forward-port-19223 branch July 3, 2023 16:14
kbuzzard pushed a commit that referenced this pull request Jul 6, 2023
There are slight differences between `mathlib3` and `mathlib4` (different set of attributes, different lemmas are in core/std), so I redid the same refactor instead of forward-porting changes.

mathlib3 PR: leanprover-community/mathlib3#19223
kim-em pushed a commit that referenced this pull request Aug 14, 2023
There are slight differences between `mathlib3` and `mathlib4` (different set of attributes, different lemmas are in core/std), so I redid the same refactor instead of forward-porting changes.

mathlib3 PR: leanprover-community/mathlib3#19223
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). mathlib3-pair This PR is a forward-port of a mathlib3 PR or part of one, either under review or recently merged ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants