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] - chore(measure_theory/integral/set_integral): update old lemmas that were in comments at the end of the file #9111

Closed
wants to merge 15 commits into from

Conversation

RemyDegenne
Copy link
Collaborator

The file set_integral had a list of lemmas in comments at the end of the file, which were written for an old implementation of the set integral. This PR deletes the comments, and adds the corresponding results when they don't already exist.

The lemmas set_integral_congr_set_ae and set_integral_mono_set are also moved to relevant sections.


Open in Gitpod

@RemyDegenne RemyDegenne added the awaiting-review The author would like community review of the PR label Sep 9, 2021
@github-actions github-actions bot added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Sep 9, 2021
@github-actions github-actions bot added merge-conflict Please `git merge origin/master` then a bot will remove this label. and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels Sep 11, 2021
@github-actions
Copy link

🎉 Great news! Looks like all the dependencies have been resolved:

💡 To add or remove a dependency please update this issue/PR description.

Brought to you by Dependent Issues (:robot: ). Happy coding!

@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Sep 11, 2021
Copy link
Collaborator

@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+
Thanks!

src/measure_theory/integral/set_integral.lean Outdated Show resolved Hide resolved
@bors
Copy link

bors bot commented Sep 13, 2021

✌️ RemyDegenne 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 The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Sep 13, 2021
@RemyDegenne
Copy link
Collaborator Author

bors r+

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Sep 13, 2021
@RemyDegenne
Copy link
Collaborator Author

bors r+

bors bot pushed a commit that referenced this pull request Sep 14, 2021
…ere in comments at the end of the file (#9111)

The file `set_integral` had a list of lemmas in comments at the end of the file, which were written for an old implementation of the set integral. This PR deletes the comments, and adds the corresponding results when they don't already exist.

The lemmas `set_integral_congr_set_ae` and `set_integral_mono_set` are also moved to relevant sections.



Co-authored-by: RemyDegenne <Remydegenne@gmail.com>
@bors
Copy link

bors bot commented Sep 14, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(measure_theory/integral/set_integral): update old lemmas that were in comments at the end of the file [Merged by Bors] - chore(measure_theory/integral/set_integral): update old lemmas that were in comments at the end of the file Sep 14, 2021
@bors bors bot closed this Sep 14, 2021
@bors bors bot deleted the set_integral_cleanup branch September 14, 2021 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants