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

Remove outdated note about hooks from documentation #3789

Merged
merged 4 commits into from Nov 5, 2022
Merged

Remove outdated note about hooks from documentation #3789

merged 4 commits into from Nov 5, 2022

Conversation

Pandapip1
Copy link
Contributor

@Pandapip1 Pandapip1 commented Nov 1, 2022

Fixes N.A.

This removes an outdated and now incorrect statement about hooks.

PR Checklist

  • Tests (N.A.)
  • Documentation
  • Changelog entry

Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

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

Thank you!

@frangio frangio enabled auto-merge (squash) November 4, 2022 16:50
@Pandapip1
Copy link
Contributor Author

@frangio the codecov isn't running for some reason - mind re-running those checks?

@frangio frangio disabled auto-merge November 5, 2022 18:13
@frangio frangio merged commit 634710d into OpenZeppelin:master Nov 5, 2022
@frangio
Copy link
Contributor

frangio commented Nov 5, 2022

Codecov can miss some updates sporadically. This PR is just docs anyway so no need to rerun it.

@Pandapip1 Pandapip1 deleted the patch-1 branch November 5, 2022 20:26
frangio pushed a commit that referenced this pull request Nov 7, 2022
Co-authored-by: Francisco <frangio.1@gmail.com>
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
(cherry picked from commit 634710d)
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.

None yet

3 participants