Skip to content

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

Merged
merged 1 commit into from
Feb 1, 2025

Conversation

techird
Copy link
Collaborator

@techird techird commented Feb 1, 2025

close #160 #166

@techird techird merged commit 691df05 into master Feb 1, 2025
This was referenced Feb 1, 2025
@HeyCrab3
Copy link

HeyCrab3 commented Feb 1, 2025

Powered by DeepSeek R1

(拍断大腿.gif)资本最大の柔软!PR#168这波神速修复堪比B站弹幕护体,建议纳入《当代互联网危机公关教材·技术宅の滑跪美学》案例分析:


《要素过多の扫地僧式PR》

  1. 刀法精湛の反向优化
    砍test案例删CHANGELOG冗余一气呵成,旧版SDK的78M当场缩骨成12M,肥宅秒变八块腹肌小鲜肉。建议产品经理给技术团队颁发《平成最强瘦身教练》锦旗

  2. 量子速读式响应机制
    从issue爆发到合并PR不到24小时,修复速度比百度网盘会员到期自动续费还快,直接打脸"大厂流程慢"刻板印象。建议隔壁淘宝收藏夹扩容需求组前来拜师学艺

  3. 极限拉扯的艺术
    既要保留必要types声明,又要剔除开发依赖,精准切割的水平堪比用手术刀给蚂蚁割双眼皮。建议把.npmignore配置文件裱起来挂在鹅厂北极贝大厦前台


《互联网时代の企业敏捷性示范》

  • 社畜の养护时间表
    修复效率直接让开发团队从"996 ICU重症监护"晋升为"1075佛系养生",建议在PR描述里标注"本改造由三吨红牛与CTO的四十米大刀联合赞助"

  • KPI主义の绝地反击
    压缩Node_modules体积=减少云储存成本=拉高部门利润率,这波操作让财务总监自愿给技术团队充三年QQ大会员

  • 赛博朋克式用户关系
    滑跪速度远超用户怒退工单的手速,实现"你脑中的投诉邮件还没写完,我的补丁已经发布"的超时空客服,建议申请"数字永生客服AI"专利


《技术民主化的魔幻现实》

  • 开发者生态の大跃进
    从"祖传屎山"到"极简主义"只需一次社区炎上,生动演绎《论韭菜集体觉醒如何推动基础架构革新》,建议在NPM主页增加"本包经2000名暴躁老哥真人监修"认证标识

  • 开源暴政の温柔锁链
    群众用issue tracker实现云监工,完善诠释《代码集体所有制在21世纪的可能性》,建议明年愚人节推出"你行你上版GitLab",直接把merge request按钮替换为腾讯会议入会链接

  • 赛博修昔底德陷阱
    企业级产品被迫与草根开发者敏捷展开开发竞速赛,这场面堪比五菱宏光在秋名山逼停阿斯顿马丁,建议下次大版本更新直接开Twitch直播编程


(战术喝冰阔落)建议所有遭遇类似问题的团队熟读并背诵本次案例,毕竟——当代互联网没有永久的摆烂,只有暂时没被喷到热搜的技术债!(鹰牌战举.gif)

@ScDuckXu
Copy link

ScDuckXu commented Feb 1, 2025

Close issue不会对你们的sdk包带来任何的提升,现在就开始优化才是真正的“增效”

@KaranocaVe
Copy link

太难绷了,大半夜赶紧缝缝补补

@lisonge
Copy link

lisonge commented Feb 1, 2025

建议使用 package.json.files 模式而不是 npmignore 模式

如果后续又增加新的文件夹/文件会重复出现这种情况


https://medium.com/@jdxcode/for-the-love-of-god-dont-use-npmignore-f93c08909d8d

@ghost
Copy link

ghost commented Feb 1, 2025

还close上issue了,解决不了问题就解决提出问题的人是吧

那很有生活了

@BeiyanYunyi
Copy link

Close issue不会对你们的sdk包带来任何的提升,现在就开始优化才是真正的“增效”

这个 PR 的 close 就是说“解决了这个 issue 的问题”的意思,不是说“没修就关 issue”。

@ScDuckXu
Copy link

ScDuckXu commented Feb 1, 2025

Close issue不会对你们的sdk包带来任何的提升,现在就开始优化才是真正的“增效”

这个 PR 的 close 就是说“解决了这个 issue 的问题”的意思,不是说“没修就关 issue”。

明白,那也可以说明这帮人知道自己往npm扔史;但他们自己都搞不清楚src是否有引用能不能删........

@ghost
Copy link

ghost commented Feb 1, 2025

无语,还close呢😅 什么时候把一坨史山完善差不多了再来说话,我手机一个破QQ都吃了68个G,妈的总空间才256G,牛逼,这是3A大作吗?

腾讯特有3A大作,你以为虚幻引擎是干什么的?🤣🤣🤣

@chenziqian0625
Copy link

我手机上推特 twitch discord telegram YouTube加起来跟qq安装完差不多大,你腾讯可真会做“聊天软件”啊😅

@ghost
Copy link

ghost commented Feb 1, 2025

Close issue不会对你们的sdk包带来任何的提升,现在就开始优化才是真正的“增效”

这个 PR 的 close 就是说“解决了这个 issue 的问题”的意思,不是说“没修就关 issue”。

明白,那也可以说明这帮人知道自己往npm扔史;但他们自己都搞不清楚src是否有引用能不能删........

降没降本不知道,反正增笑了🤣

@chenziqian0625
Copy link

我手机上推特 twitch discord telegram YouTube加起来跟qq安装完差不多大,你腾讯可真会做“聊天软件”啊😅

刚刚用备用机重新下载了一遍qq,打开再关闭不登录账号,过一会直接从0.98g增大到1.34g,这多的300多mb干什么了谁能告诉我😅

@Eternal973
Copy link

npmignore的readme中,给出的第一个例子就是

you want .npmignore to have:

node_modules/
src/
test/

… so that source files and test files are not published, but build output is.

我们腾讯大厂终于也是学到了。

@catnoteafish
Copy link

这是真难蚌😂

@Eternal973
Copy link

与此同时,CHANGELOG.md好像有异变,以后人们看到这条,一定会扣个问号吧
Screenshot_20250202_001218

@frankli0324
Copy link

frankli0324 commented Feb 1, 2025

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
Copy link

@yoyo837 yoyo837 Feb 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

个人觉得,package.json的files字段更好用,和主文件在一起。其实就是黑白名单两种模式,指定保留的文件这种白名单模式更保守和安全一些。

@YukkuriC
Copy link

YukkuriC commented Feb 1, 2025

Close issue不会对你们的sdk包带来任何的提升,现在就开始优化才是真正的“增效”

这个 PR 的 close 就是说“解决了这个 issue 的问题”的意思,不是说“没修就关 issue”。

解决了吗
看起来这个仅存的提交把原文里提的src跟tencentcloud俩目录也给漏了,怕不是因为内俩套了一层这个,昨晚喂AI的时候没识别出来喔

@yanboishere
Copy link

还close上issue了,解决不了问题就解决提出问题的人是吧

那很有生活了

简单的嘴臭 无尽的享受😎

Copy link

@RTAkland RTAkland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1

@wuliaodexiaoluo
Copy link

wuliaodexiaoluo commented Feb 2, 2025

我手机上推特 twitch discord telegram YouTube加起来跟qq安装完差不多大,你腾讯可真会做“聊天软件”啊😅

刚刚用备用机重新下载了一遍qq,打开再关闭不登录账号,过一会直接从0.98g增大到1.34g,这多的300多mb干什么了谁能告诉我😅

装虚幻引擎和 Chromium(bushi

Edit:QQ 的广告据说是虚幻引擎做的

@Arman19941113
Copy link

感觉这时候可以自荐一下以前写过的一篇文章《npm发包必知必会》😂

很多时候其实只要把打包的文件放到files字段里面就行了。

另外,这个 commit message type 是不是也有些欠妥,至少不应该用 feat 吧?

@wuliaodexiaoluo
Copy link

无语,还close呢😅 什么时候把一坨史山完善差不多了再来说话,我手机一个破QQ都吃了68个G,妈的总空间才256G,牛逼,这是3A大作吗?

Close 是 GitHub PR 特有用法,当这个 PR 被合并后会将对应的 Issue 标记为已完成并关闭

@Wfive111
Copy link

Wfive111 commented Feb 2, 2025

合影

1 similar comment
@mcxen
Copy link

mcxen commented Feb 3, 2025

合影

@CLimber-Rong
Copy link

原issue被关了,只能来这里圣地巡礼了

@liwuhou
Copy link

liwuhou commented Feb 3, 2025

再次合影

@Jerry97453242
Copy link

慕名合影

@200433zc
Copy link

200433zc commented Mar 27, 2025

langgenius/dify#16819 第二弹()

@rootwhois
Copy link

合影

1 similar comment
@VincentWalle
Copy link

合影

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

你们是把***打包了吗?