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

docs: minor updates for clarity #61

Merged
merged 2 commits into from
Jan 15, 2024

Conversation

johnsonstephan
Copy link
Contributor

@johnsonstephan johnsonstephan commented Jan 9, 2024

Purpose of this Pull Request

I've made a few minor edits to improve the clarity of the documentation.

Additional Context

I've ensured that the formatting and structure of these entries are consistent with the other items in the document.

If any further changes or adjustments are needed, please let me know, and I'll be happy to make them.

Copy link

vercel bot commented Jan 9, 2024

@johnsonstephan is attempting to deploy a commit to the Cyfrin Team on Vercel.

A member of the Team first needs to authorize it.

@johnsonstephan
Copy link
Contributor Author

Hi @PatrickAlphaC, I've submitted a PR with minor updates to the written version of the "what is an audit" from section 2 -- 2 (2-audit/1-what-is-an-audit).

Would you be the best person to review this?

@johnsonstephan
Copy link
Contributor Author

Hi @solhosty, @ShivamDeshmukh21, @Equious - I noticed that you all have been maintaining the repo and perhaps are better positioned than @PatrickAlphaC to review this PR?

Please let me know if you may have any questions. Thanks.

@PatrickAlphaC
Copy link
Member

Thanks for this! Sorry on our delay. I'll bring this up!

@solhosty
Copy link
Contributor

hey @johnsonstephan! I'll be looking at this today and doing a sync with a few other PRs here!

Copy link
Contributor

@solhosty solhosty left a comment

Choose a reason for hiding this comment

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

This looks great! Thank you for contributing! Feel free to tag me in the future with any other changes made 🙏🏻

@solhosty solhosty merged commit 63aa126 into Cyfrin:main Jan 15, 2024
1 check failed
@johnsonstephan
Copy link
Contributor Author

@PatrickAlphaC - No problem, I'm sure things are busy with the start of the year!
@solhosty - Thank you! And I'll be sure to tag you next time 🙏🏾

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.

3 participants