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(Data/Finset): golf #9058

Closed
wants to merge 1 commit into from

Conversation

urkud
Copy link
Member

@urkud urkud commented Dec 14, 2023

Golf the proof of Finset.subset_image₂ by reusing Finset.subset_image_iff.


Should it be subset_image₂ or subset_image2?
Or should we rename Set.image2 to Set.image₂?

Open in Gitpod

Golf the proof of `Finset.subset_image₂` by reusing `Finset.subset_image_iff`.
Should it be `subset_image₂` or `subset_image2`?
Or should we rename `Set.image2` to `Set.image₂`?
@urkud urkud added awaiting-review The author would like community review of the PR easy < 20s of review time. See the lifecycle page for guidelines. t-logic Logic (model theory, set theory, etc) labels Dec 14, 2023
Copy link
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors merge

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Dec 15, 2023
@eric-wieser
Copy link
Member

Should it be subset_image₂ or subset_image2? Or should we rename Set.image2 to Set.image₂?

cc @YaelDillies

mathlib-bors bot pushed a commit that referenced this pull request Dec 15, 2023
Golf the proof of `Finset.subset_image₂` by reusing `Finset.subset_image_iff`.
@YaelDillies
Copy link
Collaborator

I'm a bit split on the matter. Neither name really conveys the meaning properly. Maybe we should rename both to something likebinImage? No weird unicode and a clear meaning.

@mathlib-bors
Copy link

mathlib-bors bot commented Dec 15, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(Data/Finset): golf [Merged by Bors] - chore(Data/Finset): golf Dec 15, 2023
@mathlib-bors mathlib-bors bot closed this Dec 15, 2023
@mathlib-bors mathlib-bors bot deleted the YK-finset-subset-image2 branch December 15, 2023 13:12
awueth pushed a commit that referenced this pull request Dec 19, 2023
Golf the proof of `Finset.subset_image₂` by reusing `Finset.subset_image_iff`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy < 20s of review time. See the lifecycle page for guidelines. ready-to-merge This PR has been sent to bors. t-logic Logic (model theory, set theory, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants