git-ignore, but publish dist/ #53

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.

Sign up for GitHub

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

Jump to bottom

Merged

andywer merged 4 commits into master from feature/gitignore-but-publish-dist

May 9, 2017

Owner

andywer commented May 9, 2017 •

edited

Loading

Don't include dist/ in git repo anymore (=> fewer merge conflicts, cleaner diffs)
Still publish dist/ to npm
Add prepublish script to make sure we have an up-to-date dist/ before publishing
Change <script> usage in README (suggest using unpkg.com, since it serves npm-published files)

Andy Wermke added 3 commits

May 9, 2017 16:37


          Add prepublish script

f1398c2


          Git-ignore dist/*, change <script> url to unpkg.com

5df3f18


          Run gulp dist test on publish *only* (not on npm install)

ddbfcc4

andywer force-pushed the feature/gitignore-but-publish-dist branch from 9b349ef to ddbfcc4 Compare

May 9, 2017 14:39


          Testing on node 4 & latest is enough

fbfbef0

andywer merged commit 3a726ea into master

andywer deleted the feature/gitignore-but-publish-dist branch

May 9, 2017 14:56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet