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

[audit2] #2 Typos and Documentation #72

Merged
merged 5 commits into from
Jul 22, 2024
Merged

Conversation

stevieraykatz
Copy link
Collaborator

From spearbit:

Typos & Documentation
Status: New
Severity: Informational

created on Jul 19, 2024 at 02:56

edited by AkshaySrivastav on Jul 22, 2024 at 06:34

Description
Consider fixing the following typos and documentation issues.

@stevieraykatz stevieraykatz changed the base branch from main to Spearbit-second-audit July 22, 2024 18:49
@stevieraykatz stevieraykatz merged commit 0aa3e00 into Spearbit-second-audit Jul 22, 2024
4 of 5 checks passed
stevieraykatz added a commit that referenced this pull request Jul 24, 2024
* [audit2] #2 Typos and Documentation (#72)

* Fix typo in PaymentReceiverUpdated

* Fix typo in L2Resolver commment

* Fix typos in L2Resolver and ReverseRegistrar natspec

* Fix comment in IPriceOracle

* Fix natspec in controllers

* Remove unused event (#71)

* [audit2] #4 Cleanup GRACE_PERIOD handling for launch auction (#73)

* Cleanup GRACE_PERIOD handling for launch auction

* Add GRACE_PERIOD to EARegistrarController

* [audit2] #5 Explicitly declare state visibility (#75)

* Explicitly declare state visibility

* Style guide

* Change resolver for reverse node when changing default resolver (#76)

* Remove reverse claiming from EARegistrarController (#78)

* Add IExtendedResolver supported signal for L2Resolver (#77)

* Add reverse node claim to L2 Resolver (#81)

* [audit2] #6 Remove addr.reverse setting from ReverseRegistrar (#74)

* Remove addr.reverse setting from ReverseRegistrar

* Address PR suggestion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants