-
Notifications
You must be signed in to change notification settings - Fork 145
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
adds ability to remove contribution types when editing existing user #94
adds ability to remove contribution types when editing existing user #94
Conversation
Codecov Report
@@ Coverage Diff @@
## master #94 +/- ##
==========================================
+ Coverage 53.54% 54.43% +0.89%
==========================================
Files 19 19
Lines 409 417 +8
Branches 66 68 +2
==========================================
+ Hits 219 227 +8
Misses 161 161
Partials 29 29
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this looks good though I'd like another reviewer on this one. Also, please remove the .cli.js.swp
file. You should probably include that in your .gitignore_global
file :)
Thanks for the tip @kentcdodds. |
Hey @kentcdodds this (and #96) has been sitting idle for a good bit now. Is there anyone I can ping to get these reviewed? |
* docs: wip * Update README.md * delete files * docs: readme tweaks * motivation * motivation
What:
Adds ability to remove contribution types from the command line
Why:
Provides an easy way to update existing contribution types
How:
Mostly, updates
formatContributions()
insrc/contributors/add.js
to check for similarities/differences between theexisting
user's types and the newtypes
that are coming in and then either keep, add, or remove types.Checklist:
I kept getting formatting updates after the linting scripts ran. Let me know if these are not desireable as I tried deleting my own global
.eslintrc
config but kept getting formatting updates once the script ran regardless.