Skip to content

Conversation

muffinresearch
Copy link
Contributor

@muffinresearch muffinresearch commented Jul 6, 2017

Add contributing markdown file (based on the addons-server one with a few small changes)

Copy link
Contributor

@kumar303 kumar303 left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks. I'll make a note to also consolidate this as part of https://github.com/mozilla/addons-frontend/issues/2223

Copy link
Contributor

@tofumatt tofumatt left a comment

Choose a reason for hiding this comment

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

r+wc

CONTRIBUTING.md Outdated
Contributing
============

If you’re not sure this is the correct place to file an issue then please file an issue on the [mozilla/addons] project instead.
Copy link
Contributor

Choose a reason for hiding this comment

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

If you’re not sure this is the correct place to file an issue then please file an issue on the [mozilla/addons] project instead.

Could use with some more context. The first paragraph should provide more info:

This is the code for the Mozilla Add-ons website frontend–it's the code that displays Add-ons to user from the Add-ons (API) Server. This project handles the display of data and submission of things like reviews to the API, but doesn't handle the data. If you’re not sure this is the correct place to file an issue then please file an issue on the [mozilla/addons] project instead.

CONTRIBUTING.md Outdated

If you’re not sure this is the correct place to file an issue then please file an issue on the [mozilla/addons] project instead.

Before contributing code, please note:
Copy link
Contributor

Choose a reason for hiding this comment

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

"Before contributing" should suffice, I'd remove the word "code".

CONTRIBUTING.md Outdated

Before contributing code, please note:

- You agree to license your contributions under the [license].
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought we mostly used * for lists, but it's just a nit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah that's pandoc's fault :) - I converted this from rst here: https://github.com/mozilla/addons-server/blob/master/CONTRIBUTING.rst

CONTRIBUTING.md Outdated
Before contributing code, please note:

- You agree to license your contributions under the [license].
- Please ask on the [dev-addons mailing list] before submitting pull-requests for new features or large changes that are not related to existing issues.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would say "Please file issues or ask on the dev-addons list". No reason the discussion can't happen on here.

CONTRIBUTING.md Outdated

If you come across a potential `good first bug` for contributors, please tag it with `maybe good first bug`. The community team [triages] these every other week to ensure they have mentors assigned, onboarding information, and basic steps to get started. This gives new contributors a better experience when they pick a “good first bug” to work on.


Copy link
Contributor

Choose a reason for hiding this comment

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

Could remove a few of these empty lines.

@muffinresearch muffinresearch merged commit bebdfec into master Jul 6, 2017
@muffinresearch muffinresearch deleted the muffinresearch-contributing-1 branch July 6, 2017 16:45
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