-
Notifications
You must be signed in to change notification settings - Fork 162
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 with links/specs of style guide #11
Comments
These are in a somewhat different vein, but I would also introduce a couple suggestions for style that are mostly aimed at having readable diffs and helping the review process focus on content:
|
We are most likely going to adhere to http://www.cirosantilli.com/markdown-style-guide/. I need to make remark-lint to cooperate first. |
Our contributing guide should also include more general information-- particularly on getting started with GitHub, since many contributors may be joining for the first time ! |
+1 to @emdupre's point. Feel free to grab anything from the starter kit that make sense! https://github.com/bids-standard/bids-starter-kit/blob/master/CONTRIBUTING.md |
This is also a good tutorial: https://egghead.io/lessons/javascript-introduction-to-github |
Hey folks in the thread - I've been mostly offline for a few day - is anyone working on the contributors guide? I can take the lead if that's helpful....I might need a week or so to get it done but happy to be assigned unless anyone else wants to?! (tagging @park-patrick as he did lots of work with me on the |
Thanks for offering to help! We definitely need more documentation for contributors. Happy to "assign" you to this issue, but you will need to accept the invitation to join |
Done! Thank you - I'd missed that :) |
Done! |
Hi @KirstieJane, sure I'd be open to helping! @chrisfilo could you send me an invitation as well? |
@park-Patrick - I don’t think you need to be a member. It makes sense to do the work in a pull request. I only need to be part of the organisation to be “assigned” the issue (and for other reasons - that’s why Chris had sent the invite a week ago!) If you have time to start pulling across some of the useful points from the starter kit that would be great! |
started the pull request here! #44 |
REF: initial rewriting of coordinate system keys
I realized as I was reading through the raw specification file that I was hesitant to suggest style edits because I didn't know if any decisions had been made about which flavor of markdown we are using, so I didn't know how to suggest changes so things like tables and file paths/directory trees.
I contributing guide with a record of markdown formatting decisions would be useful.
Specifically, the things I wondered about right away were:
tree
outputs?) of just bullet-ed/tabbed?The text was updated successfully, but these errors were encountered: