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

[R4R] - {develop}: fix consensys audit issue: cs-6.18 #1243

Merged
merged 3 commits into from
Jul 4, 2023

Conversation

abelliumnt
Copy link
Collaborator

Goals of PR

Core changes:

  • change variable type and add value checker

Notes:

  • Write notes here

Related Issues:

@abelliumnt abelliumnt changed the title [WIP] - {develop}: fix consensys audit issue: cs-6.18 [R4R] - {develop}: fix consensys audit issue: cs-6.18 Jul 3, 2023
@abelliumnt abelliumnt requested review from byteflyfunny, Sha3nS and wukongcheng and removed request for byteflyfunny July 3, 2023 11:57
guoshijiang
guoshijiang previously approved these changes Jul 4, 2023
byteflyfunny
byteflyfunny previously approved these changes Jul 4, 2023
@abelliumnt abelliumnt dismissed stale reviews from byteflyfunny and guoshijiang via 58b863f July 4, 2023 09:27
@byteflyfunny byteflyfunny merged commit 8b03b1b into develop Jul 4, 2023
1 of 6 checks passed
@byteflyfunny byteflyfunny deleted the lhy/fix-cs18 branch July 4, 2023 09:28
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.

None yet

3 participants