-
Notifications
You must be signed in to change notification settings - Fork 112
feat: 优化 npm 分发包内容,不再分发示例、用例和变更日志 #168
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
Powered by DeepSeek R1(拍断大腿.gif)资本最大の柔软!PR#168这波神速修复堪比B站弹幕护体,建议纳入《当代互联网危机公关教材·技术宅の滑跪美学》案例分析: 《要素过多の扫地僧式PR》
《互联网时代の企业敏捷性示范》
《技术民主化的魔幻现实》
(战术喝冰阔落)建议所有遭遇类似问题的团队熟读并背诵本次案例,毕竟——当代互联网没有永久的摆烂,只有暂时没被喷到热搜的技术债!(鹰牌战举.gif) |
Close issue不会对你们的sdk包带来任何的提升,现在就开始优化才是真正的“增效” |
太难绷了,大半夜赶紧缝缝补补 |
建议使用 package.json.files 模式而不是 npmignore 模式 如果后续又增加新的文件夹/文件会重复出现这种情况 https://medium.com/@jdxcode/for-the-love-of-god-dont-use-npmignore-f93c08909d8d |
还close上issue了,解决不了问题就解决提出问题的人是吧 那很有生活了 |
这个 PR 的 close 就是说“解决了这个 issue 的问题”的意思,不是说“没修就关 issue”。 |
明白,那也可以说明这帮人知道自己往npm扔史;但他们自己都搞不清楚src是否有引用能不能删........ |
腾讯特有3A大作,你以为虚幻引擎是干什么的?🤣🤣🤣 |
我手机上推特 twitch discord telegram YouTube加起来跟qq安装完差不多大,你腾讯可真会做“聊天软件”啊😅 |
降没降本不知道,反正增笑了🤣 |
刚刚用备用机重新下载了一遍qq,打开再关闭不登录账号,过一会直接从0.98g增大到1.34g,这多的300多mb干什么了谁能告诉我😅 |
在npmignore的readme中,给出的第一个例子就是
我们腾讯大厂终于也是学到了。 |
这是真难蚌😂 |
提一嘴,npmignore这个 package 并不是.npmignore文件的“官方介绍”,而是用户编写的用于编辑这一文件的工具,其readme不具有指导意义 the correct place to go is npm docs 注意到,即使将CHANGELOG放进.npmignore里,其也不会被ignore,即使不将node modules放进去,其也会被ignore 再注意到,npm更推荐使用package.json的files字段而非npmignore来控制发布的 又,各位真正写过node的大伙,不会真的以为node modules里的所有东西都会被bundle到dist里吧,还是说各位都是在生产环境里npm run dev的凤雏? disclaimer:利益无关 |
examples/ | ||
test/ | ||
CHANGELOG.md | ||
SERVICE_CHANGELOG.md |
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.
个人觉得,package.json的files字段更好用,和主文件在一起。其实就是黑白名单两种模式,指定保留的文件这种白名单模式更保守和安全一些。
解决了吗 |
简单的嘴臭 无尽的享受😎 |
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.
1
装虚幻引擎和 Chromium(bushi Edit:QQ 的广告据说是虚幻引擎做的 |
感觉这时候可以自荐一下以前写过的一篇文章《npm发包必知必会》😂 很多时候其实只要把打包的文件放到 另外,这个 commit message type 是不是也有些欠妥,至少不应该用 feat 吧? |
Close 是 GitHub PR 特有用法,当这个 PR 被合并后会将对应的 Issue 标记为已完成并关闭 |
合影 |
1 similar comment
合影 |
原issue被关了,只能来这里圣地巡礼了 |
再次合影 |
慕名合影 |
langgenius/dify#16819 第二弹() |
合影 |
1 similar comment
合影 |
close #160 #166