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

Update to 1.82 #529

Closed
wants to merge 16 commits into from
Closed

Update to 1.82 #529

wants to merge 16 commits into from

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Oct 31, 2024

No description provided.

@Veykril Veykril force-pushed the veykril/push-pxyquklquyur branch from ab3c333 to 0e784c5 Compare December 3, 2024 11:13
Hoverbear
Hoverbear previously approved these changes Dec 3, 2024
Copy link
Member

@Hoverbear Hoverbear left a comment

Choose a reason for hiding this comment

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

Looks good to me

@Veykril Veykril force-pushed the veykril/push-pxyquklquyur branch from 5823ae2 to 7a8d9ac Compare December 5, 2024 11:12
@Veykril Veykril marked this pull request as ready for review December 5, 2024 11:12
src/glossary.rst Outdated Show resolved Hide resolved
src/patterns.rst Outdated
@@ -570,7 +567,7 @@ A :t:`reference pattern` is a :t:`pattern` that dereferences a :t:`pointer` that
is being matched.

:dp:`fls_x0bmzl1315gq`
A :t:`reference pattern` is an :t:`irrefutable pattern`.
A :t:`reference pattern` is an :t:`irrefutable pattern` when its nested :t:`pattern` is an :t:`irrefutable pattern`.
Copy link
Member

Choose a reason for hiding this comment

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

I don't understand this sentence

Copy link
Member Author

Choose a reason for hiding this comment

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

This is meant to state that the irrefutable pattern-ness is "inherited". So &Pattern is irrefutable when Pattern is irrefutable. Any ideas for how to phrase this better? I guess the use of "nested" here is the confusing part?

Copy link
Member

@tshepang tshepang Dec 9, 2024

Choose a reason for hiding this comment

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

oh, maybe emphasizing is would make it more clear...

A reference pattern is an irrefutable pattern when its nested pattern is an irrefutable pattern

otherwise it's very clear after you explain it, so feel free to ignore this suggestion

Copy link
Member

Choose a reason for hiding this comment

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

alternative to emphasizing "is" is adding "itself"

A reference pattern is an irrefutable pattern when its nested pattern is itself an irrefutable pattern

Copy link
Member Author

Choose a reason for hiding this comment

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

Added itself to it

src/types-and-traits.rst Outdated Show resolved Hide resolved
@Veykril Veykril force-pushed the veykril/push-pxyquklquyur branch from 7a8d9ac to 6f37305 Compare December 9, 2024 11:50
@Veykril Veykril force-pushed the veykril/push-pxyquklquyur branch from 6f37305 to 3055f43 Compare December 9, 2024 12:57
@Veykril Veykril mentioned this pull request Dec 9, 2024
@tshepang
Copy link
Member

tshepang commented Dec 9, 2024

made obsolete by #536

@tshepang tshepang closed this Dec 9, 2024
@tshepang tshepang deleted the veykril/push-pxyquklquyur branch December 9, 2024 13:13
bors-ferrocene bot added a commit that referenced this pull request Dec 10, 2024
536: Update to 1.82 and 1.83 r=tshepang a=Veykril

Builds atop of #529 

Co-authored-by: Lukas Wirth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants