-
Notifications
You must be signed in to change notification settings - Fork 12.2k
Group typo fixes #5466
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
Group typo fixes #5466
Conversation
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Arr00 <[email protected]>
|
Name | Type |
---|---|
openzeppelin-solidity | Minor |
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
|
||
# Disclaimer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've been against changing archived audits but if consensus is we should change it, that's fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think there is consensus. Just grouping everything under one PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree we shouldn't change the audits. However, in this case I'm starting to lean towards changing it just because it's the most pragmatic decision to avoid having more bot PRs in the future. We've already closed and discussed this a couple of times.
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Hadrien Croubois <[email protected]>
Co-authored-by: Bilog WEB3 <[email protected]> Co-authored-by: Fallengirl <[email protected]> Co-authored-by: Hadrien Croubois <[email protected]> Co-authored-by: XxAlex74xX <[email protected]> Co-authored-by: Arr00 <[email protected]> Co-authored-by: PixelPilot <[email protected]> Co-authored-by: kilavvy <[email protected]> Co-authored-by: Devkuni <[email protected]> Co-authored-by: Danbo <[email protected]> Co-authored-by: Ann Wagner <[email protected]> Co-authored-by: comfsrt <[email protected]> Co-authored-by: Bob <[email protected]> Co-authored-by: JohnBonny <[email protected]> Co-authored-by: moonman <[email protected]> Co-authored-by: kazak <[email protected]> Co-authored-by: Wei <[email protected]> Co-authored-by: Maxim Evtush <[email protected]> Co-authored-by: Vitalyr <[email protected]> Co-authored-by: pendrue <[email protected]> Co-authored-by: Tronica <[email protected]> Co-authored-by: emmmm <[email protected]> Co-authored-by: bigbear <[email protected]> Co-authored-by: Tomás Andróil <[email protected]> Co-authored-by: GooseMatrix <[email protected]> Co-authored-by: jasmy <[email protected]> Co-authored-by: SITADRITA1 <[email protected]> Co-authored-by: Ocenka <[email protected]> Signed-off-by: Hadrien Croubois <[email protected]>
The #typo-fixes branches accumulated typo fixes. This branch should be merged before freezing a release candidate.
This is done to avoid multiple small commits on that polute the master branch. All typo fix PR should target this branch.