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 contribution section to README #3

Merged
merged 1 commit into from
Oct 27, 2016
Merged

Conversation

smudge
Copy link
Member

@smudge smudge commented Oct 27, 2016

/domain @samandmoore @jmileham
resolves #1. removed the part about making sure tests are passing... 🤔

resolves Betterment#1. removed the part about making sure tests are passing... 🤔
@nanda-prbot
Copy link

@smudge needs to request domain (and maybe platform) reviewers.

@jmileham
Copy link
Member

/domain @jmileham @samandmoore
<< domain lgtm!

Yeah, jshint is the only thing between the CRX and the abyss right now. Added an issue around that: #4

@nanda-prbot
Copy link

Needs somebody from @jmileham and @samandmoore to claim domain review.

Use the shovel operator to claim, e.g.:

@myname << domain && platform

@jmileham jmileham merged commit 2164012 into Betterment:master Oct 27, 2016
@smudge smudge deleted the patch-1 branch October 27, 2016 16:55
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.

Add a "Contributing" section to the README
3 participants