Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Add hex int pattern entry to CONTRIBUTING #279

Merged

Conversation

ericnordelo
Copy link
Member

@ericnordelo ericnordelo commented Nov 8, 2022

Fixes #216

andrew-fleming
andrew-fleming previously approved these changes Nov 8, 2022
Copy link
Contributor

@andrew-fleming andrew-fleming left a comment

Choose a reason for hiding this comment

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

Very nice, @ericnordelo! I left a couple minor suggestions, but I think we're looking good :)

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Co-authored-by: Andrew Fleming <[email protected]>
Co-authored-by: Andrew Fleming <[email protected]>
andrew-fleming
andrew-fleming previously approved these changes Nov 8, 2022
Copy link
Contributor

@andrew-fleming andrew-fleming 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!

martriay
martriay previously approved these changes Nov 10, 2022
CONTRIBUTING.md Outdated Show resolved Hide resolved
@martriay martriay dismissed stale reviews from andrew-fleming and themself via 745c632 November 10, 2022 22:15
@martriay martriay merged commit 66eef0c into OpenZeppelin:main Nov 10, 2022
@ericnordelo ericnordelo deleted the feat/document-hex-int-pattern-#216 branch November 10, 2022 23:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document hex/int pattern
3 participants