-
-
Notifications
You must be signed in to change notification settings - Fork 45
Use files
property in package.json
#7
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
Conversation
+1 test/out.json is over 250kb. |
Use `files` property in package.json
Thanks to both of you (also for #8)! |
@fb55 No problem! I'd be glad if you release a new version :) |
Apparently @npm has problems with the name again: npm/npm#4665. I can't publish a new version right now, but would be willing to switch the package to a lowercase name (if old references could be redirected). cc @othiym23 |
Oh, I didn't know that problem. |
@fb55 Existing packages should be fine. You can push updates to them. you just can't create NEW packages with uppercase letters in the name, afaik. If you want to lowecase it, you can soft-redirect with the |
@isaacs This is the response I get:
edit: My point is that it doesn't seem to work; I'm aware that the log isn't especially helpful. |
@fb55 Yeah, I get it. I see, it's validating before sending to the registry. That feels like a reasonably-motivated idea that is unfortunately causing some unpleasant behavior in this case. We should be rejecting new package creation with uppercase names, but not updates to existing packages. I'll bring it up with the team here. |
Thanks @isaacs! |
No description provided.