Skip to content

Add contribution rules#10

Merged
HyperWinX merged 3 commits intomasterfrom
how-to-contribute
Apr 7, 2025
Merged

Add contribution rules#10
HyperWinX merged 3 commits intomasterfrom
how-to-contribute

Conversation

@HyperWinX
Copy link
Member

No description provided.

@mentatbot
Copy link

mentatbot bot commented Apr 6, 2025

I'm getting to work. See my steps and track my cost here.

  • Wake on any new activity.
    I always see new activity while actively working. When paused, I'll only respond when tagged unless this box is checked.

@mentatbot
Copy link

mentatbot bot commented Apr 6, 2025

Thanks for adding this CONTRIBUTION.md file! Having clear contribution guidelines is very helpful for new contributors.

I noticed a few things that might need addressing:

  1. There's a typo in "collabolators" - should be "collaborators"

  2. The PR instructions in this file say to create branches from and make PRs to the master branch, but the previous README specifically said:

    Please send your pull requests to dev repository branch, not master.

    Is this change intentional? If so, it might be worth mentioning this change in workflow in the PR description.

  3. Some minor suggestions for clarity:

    • In step 3 for collaborators, consider adding a period after "others"
    • The sentence in step 4 for collaborators is quite long and could be broken up for readability
    • For external contributors, you might want to add more detail about how to fork the repository

Overall, this looks like a useful addition to help guide contributors!

@HyperWinX HyperWinX merged commit 90d8496 into master Apr 7, 2025
12 of 13 checks passed
HyperWinX added a commit that referenced this pull request Apr 12, 2025
* Add CONTRIBUTION.md

* Fix typo

* Add blank lines before tables [codacy]
@HyperWinX HyperWinX deleted the how-to-contribute branch April 13, 2025 18:56
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.

1 participant