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.

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

include dist to npm #100

Merged
merged 1 commit into from
Jul 21, 2015
Merged

include dist to npm #100

merged 1 commit into from
Jul 21, 2015

Conversation

iofjuupasli
Copy link
Contributor

I use npm for AMD. qs is only library from my front-end deps which doesn't include dist in npm package.

@nlf nlf added the bug label Jul 21, 2015
@nlf nlf self-assigned this Jul 21, 2015
@nlf nlf added this to the 4.0.1 milestone Jul 21, 2015
nlf added a commit that referenced this pull request Jul 21, 2015
@nlf
Merge pull request #100 from iofjuupasli/master
ae35da1 
include dist to npm
@nlf nlf merged commit ae35da1 into ljharb:master Jul 21, 2015
@nlf nlf modified the milestones: 5.0.0, 4.0.1 Aug 27, 2015
@peril-staging peril-staging bot mentioned this pull request Jan 15, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nlf nlf

Labels
bug
Projects
None yet
Milestone
5.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@iofjuupasli @nlf