-
Notifications
You must be signed in to change notification settings - Fork 45
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
Version 2 Checklist #50
Comments
@mgred let's use this as a base right now. We can work off |
It might be good to use some of the github templating if you would like people to follow contributing guidelines. https://blog.github.com/2016-02-17-issue-and-pull-request-templates/ |
@drewjbartlett cool plan.
I like that workflow, think its quite efficient to do small but qualitative steps
You can give me any task, with a short description in an issue, I think that's enough |
Version 2 Checklist
Documentation
Fixes
Enhancements
api
renamed tohttp
apiConfig
tohttpConfig
qs
forqs-stringify
since we only use that method https://www.npmjs.com/package/qs-stringify.default
helper to go withmodel
/collection
photo.get('/tags', { order: orders.ASC })
.js
filesdeepmerge
instead of lodashwithData
towithConfig
qs-stringify
with old debuggeraxios
entirelyhttpConfig
from configdebug
from configBuild
prepublish
hook and replace with: Useprepare
for build steps andprepublishOnly
for upload-only.Tests
model
,collection
,default
I can add a contributing guide for this. But for now:
npm run unit:watch
while working on new features and fixes. Do you best to make small changes while always keeping the tests green.For now we can use issues or projects. @mgred I have not used projects for this but we can look into it!
The text was updated successfully, but these errors were encountered: