-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
add CONTRIBUTING.md #4741
add CONTRIBUTING.md #4741
Conversation
64bf7ea
to
aa9760f
Compare
aa9760f
to
1f7d46d
Compare
1f7d46d
to
b3e6f24
Compare
I think it is good to have this file, but I would prefer to keep it as concise as possible, with links to the wiki. The wiki is a better place to e.g. describe how to checkout a branch, it is also means we don't have duplicated information, and the wiki is somewhat easier to update. There is also overlap with https://stockfishchess.org/get-involved/ so ideally these two places are also in sync. |
dc4ce11
to
ae7fbb3
Compare
Looks good to me now. Two suggestions, to make the style more similar to our Readme.md file:
|
No functional change
ae7fbb3
to
3b387ba
Compare
Good suggestions, thanks. |
No functional change