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: typos in the docs #1527

Closed
wants to merge 3 commits into from

Conversation

tudorpintea999
Copy link

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

@xgreenx
Copy link
Collaborator

xgreenx commented Dec 3, 2023

Thank you, but we don't want to accept super small PRs from accounts without real Github activity

Thank you, but we don't want to accept super small PRs from accounts without long real/valuable Github activity.

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

without real activity? i am New User of the Month award for November 2023 on fuel 's forum.
I created automation scripts for running and configuring testnet nodes for other projects, venom, base, taiko.
How can you say I don t have real activity?

@xgreenx
Copy link
Collaborator

xgreenx commented Dec 4, 2023

I created automation scripts for running and configuring testnet nodes for other projects, venom, base, taiko.

You created these scripts for yourself. It is not a public contribution and can be easily simulated.

You created many small pull requests for many crypto projects. All of them are related to typoes -> You have some tool that allows you to find typos in the documentation. It would be more valuable if you create a pull request with CI that runs this tool automatically and helps developers write the right documentation instead of creating a pull request for each word.

image

Some of your contributions don't make sense, which means you are looking for the quantity of merged PRs instead of the quality of these PRs.

image

https://forum.fuel.network/t/fip-01-fuel-improvement-proposal-enhanced-parallelism-and-developer-experience/3564

This proposal looks to me as auto-generated by AI.

@tudorpintea999
Copy link
Author

taikoxyz/simple-taiko-node#142 (review)

i don t find any problem by using the latest technology as AI in order to help crypto projects deliver the best documentation. If I knew sway lang or rust I would definitely help there, but for now my knowledge is limited to documentation/nodes/bash.

@xgreenx
Copy link
Collaborator

xgreenx commented Dec 4, 2023

i don t find any problem by using the latest technology as AI in order to help crypto projects deliver the best documentation.

If you can integrate typo-checking into the CI, it will be awesome=) It is a valuable contribution and we will accept it(as any other project).

@tudorpintea999
Copy link
Author

looks like I got a challenge here........ challenge accepted !

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