Skip to content

[Merged by Bors] - chore: scope symmDiff notations#9844

Closed
PatrickMassot wants to merge 2 commits intomasterfrom
pm_scope_symmdiff
Closed

[Merged by Bors] - chore: scope symmDiff notations#9844
PatrickMassot wants to merge 2 commits intomasterfrom
pm_scope_symmdiff

Conversation

@PatrickMassot
Copy link
Copy Markdown
Member

@PatrickMassot PatrickMassot commented Jan 19, 2024

Those notations are not scoped whereas the file is very low in the import hierarchy.


Open in Gitpod

@PatrickMassot PatrickMassot added awaiting-review easy < 20s of review time. See the lifecycle page for guidelines. awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. labels Jan 19, 2024
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Jan 19, 2024
Copy link
Copy Markdown
Contributor

@sgouezel sgouezel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 19, 2024

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

@ghost ghost added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Jan 19, 2024
Copy link
Copy Markdown
Contributor

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you scope that in Order instead?

Co-authored-by: sgouezel <sebastien.gouezel@univ-rennes1.fr>
@PatrickMassot
Copy link
Copy Markdown
Member Author

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Jan 19, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jan 19, 2024
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 19, 2024

Build failed:

  • Build

@YaelDillies
Copy link
Copy Markdown
Contributor

Also please include motivation for this change. This PR is currently unjustified for someone coming across it on GitHub.

@PatrickMassot
Copy link
Copy Markdown
Member Author

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Jan 19, 2024
Those notations are not scoped whereas the file is very low in the import hierarchy.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 19, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: scope symmDiff notations [Merged by Bors] - chore: scope symmDiff notations Jan 19, 2024
@mathlib-bors mathlib-bors bot closed this Jan 19, 2024
@mathlib-bors mathlib-bors bot deleted the pm_scope_symmdiff branch January 19, 2024 20:58
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). easy < 20s of review time. See the lifecycle page for guidelines. ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants