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

fix: typos #991

Merged
merged 2 commits into from
Mar 21, 2023
Merged

fix: typos #991

merged 2 commits into from
Mar 21, 2023

Conversation

julio4
Copy link
Contributor

@julio4 julio4 commented Mar 14, 2023

Spotted some typos in comments, variable and function names while reading through the code base.

Copy link
Contributor

@rodrigo-pino rodrigo-pino left a comment

Choose a reason for hiding this comment

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

So many typos 😱! Thanks for your contribution!

Copy link
Contributor

@rjnrohit rjnrohit left a comment

Choose a reason for hiding this comment

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

Thanks!

@AlejandroLabourdette
Copy link
Contributor

Nice! could you also merge develop into this PR so it can pass all tests?.

@julio4
Copy link
Contributor Author

julio4 commented Mar 21, 2023

@AlejandroLabourdette Done !

@piwonskp piwonskp merged commit 15ef093 into NethermindEth:develop Mar 21, 2023
piwonskp pushed a commit that referenced this pull request Apr 4, 2023
* Fix cairo-lang dependencies issues after major upgrades (#995)

* fix: typos (#991)

* Bump @sideway/formula from 3.0.0 to 3.0.1 in /docs (#990)

Bumps [@sideway/formula](https://github.com/sideway/formula) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/sideway/formula/releases)
- [Commits](hapijs/formula@v3.0.0...v3.0.1)

---
updated-dependencies:
- dependency-name: "@sideway/formula"
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Franco Barpp Gomes <franco.b.gomes@gmail.com>
Co-authored-by: julio4 <30329843+julio4@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

5 participants