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

fix: several typos in the documentation #3526

Closed
wants to merge 19 commits into from

Conversation

tudorpintea999
Copy link

Hello
I fixed some typos in the documentation,
Hope it helps.
Br,
Tudor.

@ludamad
Copy link
Collaborator

ludamad commented Dec 3, 2023

Hi. We've recently decided on a policy that we will not be merging trivial PRs unless someone has done a significant contribution already. The recent usage of Github commits as a metric mean we have to manage this a bit. We will redo this change and thank you for the contribution.

@ludamad ludamad closed this Dec 3, 2023
@tudorpintea999
Copy link
Author

hello, i totally understand that, after starknet leak people started to "contribute more" to all the projects. I don t have (yet) noir or rust knowledge to contribute to the architecture of the projects that why I tried to do my best with the technical documentation. Totally understood the reason. Thanks

@ludamad
Copy link
Collaborator

ludamad commented Dec 4, 2023

Round-up here #3556

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.

2 participants