Skip to content

brief contribution tutorial#3

Open
latexdrivinggloves wants to merge 9 commits intoradio-astro-tools:masterfrom
latexdrivinggloves:master
Open

brief contribution tutorial#3
latexdrivinggloves wants to merge 9 commits intoradio-astro-tools:masterfrom
latexdrivinggloves:master

Conversation

@latexdrivinggloves
Copy link

Created first draft of a tutorial on how to contribute code, along with a demo script, helloworld.py. Also updated READMEs to reflect helloworld changes.

@keflavich
Copy link
Contributor

This is good. Let's turn the .txt into a .md- it shouldn't require much work but will look a lot nicer.

@latexdrivinggloves
Copy link
Author

Bah! I didn't mean to close this...

@astrofrog astrofrog reopened this Apr 15, 2014
@latexdrivinggloves
Copy link
Author

The contribution tutorial is a bit better formatted and now includes instructions for those using git v1.7.1 (which is evidently the version on our RHEL 6 computers and it's difficult to update due to root access issues.) Sorry for all the tiny commits at the end, I was tweaking the .md file and seeing how it looked online.

@astrofrog
Copy link
Member

@brisbind - thanks for working on this! Will review this.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a good idea to wrap the text to 78 characters so that it is guaranteed to look fine in any text editor

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, nice to make things like radioscripts_contrib into links. With rst, that is done by something like

`radioscripts_contrib <https://github.com/radio-tools/radioscripts_contrib>`_

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.

3 participants