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

add contributor guidelines #156

Merged
merged 3 commits into from
Dec 19, 2018
Merged

Conversation

danielhuppmann
Copy link
Member

@danielhuppmann danielhuppmann commented Dec 17, 2018

This PR adds a simple contributor guidelines document following the suggestion of reviewers at openjournals/joss-reviews#1095.

Are there clear guidelines for third parties wishing to

  • 1) Contribute to the software
  • 2) Report issues or problems with the software
  • 3) Seek support

@gidden
Copy link
Member

gidden commented Dec 17, 2018

Hey @danielhuppmann this looks like a great first cut. The JOSS reviewers include the following list:

Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support

Do you think we cover everything there with existing docs + this PR?

See the conversation here

@danielhuppmann
Copy link
Member Author

danielhuppmann commented Dec 17, 2018 via email

@gidden
Copy link
Member

gidden commented Dec 17, 2018

Perfect, agreed. Just made one here: https://groups.google.com/forum/#!forum/pyam-iamc

Mind updating this and then I can pull it in?

@danielhuppmann
Copy link
Member Author

added the reference to the Googlegroup in the contributor guidelines and on the documentation pages, updated the PR description to include the JOSS documentation review question (and all boxes now ticked off).

@gidden
Copy link
Member

gidden commented Dec 19, 2018

Merging in so we can try to finish off Joss stuff today.

@gidden gidden merged commit 6876b33 into IAMconsortium:master Dec 19, 2018
@danielhuppmann danielhuppmann deleted the contributing branch December 20, 2018 11:05
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