使WordPress成为核心

开的5年前

关闭14个月前

上次修改时间13个月前

#47749 关闭 增强 (固定的)

生成工具:清除时删除所有旧文件

报告人: azaozz的个人资料 阿佐兹 所有者: 阿佐兹简介 阿佐兹
里程碑: 6.2 优先级: 正常的
严重程度: 正常的 版本:
组件: 构建/测试工具 关键词: has-patch接口 has-testing-info(有测试信息)
重点: 复写的副本:

描述

当前正在运行清洁的不会删除所有旧文件。它要么只清洁/建造或仅限中的内联文件/型钢混凝土。切换建筑位置时,这可能会造成混淆。

中生成文件的存在/型钢混凝土这样做也会导致一些问题咕哝式预承诺或建筑到/建造,请参阅#47078.

附件(2)

47749.差异(23.3 KB)-由添加阿佐兹 5年前.
47749.1差异(30.9 KB)-由添加阿佐兹 5年前.

将所有附件下载为:.zip

更改历史记录(33)

@阿佐兹
5年以前

#1 @氮杂氮
5年以前

  • 关键词 需求-测试补充

在47749.diff中:修复清洁的任务从/build和/src中清除旧文件。

该修补程序还在Gruntfile.js中包含一些不相关的WPCS修复程序。

上次编辑时间5年前通过阿佐兹(以前的)(差异)

#2 @德卡法
5年以前

  • 关键词 开发人员反馈补充
正在加载“Gruntfile.js”任务。。。错误>>错误:找不到模块“@babel/runtime/helpers/interopRequireDefault”警告:未找到任务“生成”。使用--force继续。由于警告而中止。

#3个 @德卡法
5年以前

糟糕,我更新了babel运行时

npm安装@babel/runtime@最新

然后检查,补丁工作正常,它工作了:)。

上次编辑时间5年前通过德卡法(以前的)(差异)

#4 @阿佐兹
5年以前

  • 关键词 开发人员反馈远离的

@dkarfa谢谢你的测试!是的,我忘记跑步了npm安装有时也是这样。很高兴它能按预期工作。

@阿佐兹
5年以前

#5 @阿佐兹
5年以前

更多地关注了“咕哝清洁”的作用及其原因:

  • 清除:文件从中清除所有内容/建造。想知道是否需要分别清理那里的/js和/css。
  • 在构建js时运行Webpack也会创建/src/wp-includes/css/dist。清理js时需要“清理”它,否则它可能会在创建后被删除。
  • 建造至后/建造有剩菜/src/wp-includes/js包含一些js文件的目录。

在47749.1.diff中:

  • 交换机咕哝清理:js咕噜声清洁:css只在里面清洁/型钢混凝土。这取决于传球--开发但不确定是否需要。
  • 删除/src/wp-includes/css/dist运行时清除:js.
  • 运行清除:js建成后/建造要删除/src/wp-includes/js.
上次编辑时间5年前通过阿佐兹(以前的)(差异)

#6 @德斯罗什
5年以前

  • 里程碑已从更改5.3未来版本

几个小时后就有5.3 beta 1了,我要用这个。然而,因为它与测试和工具相关,所以如果有人有时间拥有它,可以在测试期间将其移回并提交。

#7 @isabel_监狱
3年以前

  • 关键词 需要-刷新补充

经过测试,补丁不再适用。

此外,WPCS的更改使得很难阅读实际代码的更改位置,所以我不能100%确定这里发生了什么。澄清:清洁的描述中提到的任务是在运行时运行的任务咕噜声建造grunt构建:开发是吗?因此,目的是确保从build/和src/中清除旧文件,无论我们从哪个目录进行构建?

上次编辑时间3年前通过isabel_监狱(以前的)(差异)

这张票是在采购订单号3843WordPress/WordPress-develop开发通过@铁编程器.


16个月以前
#9个

  • 关键词 has-patch接口补充;需要-刷新远离的

刷新以前的工作追踪47749.

Props azaozz、dkarfa、desrosj、isabel_brison、SergeyBiryukov。

Trac票:https://core.trac.wordpress.org/ticket/47749

#10 @铁编程器
16个月以前

  • 关键词 has-testing-info(有测试信息)补充
  • 里程碑已从更改未来版本6.2

测试说明

测试步骤清洁的任务

  1. 构建文字印刷-开发开发环境,具有:npm运行构建:dev。这将在型钢混凝土/目录。
  2. 通过列出不受源代码管理的文件来确认编译的文件存在:git ls文件-o src/wp-admingit ls-files-o src/wp包含。应返回大量文件。
  3. 运行npm运行咕哝清洁.
  4. 通过重新运行步骤2中的命令,确认已删除编译的文件。

测试步骤建造任务

  1. 构建文字印刷-开发分发方式:npm运行生成。这将在型钢混凝土/目录并将其复制到建造/.
  2. 确认已从型钢混凝土/通过列出不受源代码管理的文件生成后的目录:git ls文件-o src/wp-admingit ls-files-o src/wp包含。应该没有结果。

预期结果

  • 期间建造,中不应有与编译文件相关的错误src/wp-管理员src/wp包括(请参阅其他信息)。
  • 完成上述每个测试后src/wp-管理员src/wp包括.

其他信息

在开发过程中型钢混凝土/目录中可能会填充从源代码管理中排除的文件。这可能包括修补程序/合并文件等内容(.orig|.rej)甚至孤立的目录(src/wp-includes/blocks/some-removed-block文件/). 如果由于这些文件导致生成错误,则应手动解决这些错误。

上次编辑时间16个月前通过铁编程器(以前的)(差异)

@口香糖27对发表了评论公共关系编号3843:


15个月以前
#11

谢谢@ironprogrammer,按照@azaozz补丁47749.1差异有什么东西丢了吗?

'清除:文件'中缺少grunt.registerTask('build',function(){

@铁编程器对发表了评论采购订单号3843:


15个月以前
#12个

谢谢@ironprogrammer,按照@azaozz补丁47749.1差异有什么东西丢了吗?

'清除:文件'中缺少grunt.registerTask('build',function(){

你好,@mukeshpanchal27👋🏻

这个使用“build:dev”`影响型钢混凝土/目录,所以我选择保守,不修改建造/,这是分开处理的.构建:开发包括构建:js构建:css,它们各自处理各自的清洁任务。

我相信这解决了所报道的问题,但也许@azaozz可以证实🙏🏻.

#13 随访: @罗宾开发者
15个月以前

Hi@ironprogrammer在遵循测试说明之后,我看到多站点文件仍然存在。
你认为我们应该覆盖多个站点还是跳过?

感谢您的详细说明。
截图:
之前-https://d.pr/i/aWR7EX网站
之后-网址:https://d.pr/i/5qfYhh

#14 @拉泽007
15个月以前

我已经按照测试说明进行了操作。这是结果。

单个站点
清理任务:
之前:wp包括:https://d.pr/i/VAo0e8

wp-管理员:https://d.pr/i/kVpfxt

之后:wp-admin包括(&W):https://d.pr/i/bdVy2w

生成任务:
应用修补程序后:https://d.pr/i/fhwMZ网站

#15 @成本开发
15个月以前

  • 关键词 犯罪补充;需求-测试远离的

公关3843正在测试单个站点的性能。由于我们今天将发布6.2 Beta 1,我将删除需要测试做好准备犯罪帮助清理报告。由于这是一个构建/测试工具票据,可以在周期中的任何时间提交,因此可以在周期的稍后或下一个周期中处理多站点。🙂

附言:我想把这个改为Task(祝福),但关于什么时候合适的讨论仍在进行中。我将把这个决定留给@azaozz🙂

附加道具:@mukesh27

这张票是在松弛(Slack)costdev的in#core。查看日志.


15个月以前

#17个 答复: 13 @铁编程器
15个月以前

回复罗宾开发者:

Hi@ironprogrammer在遵循测试说明之后,我看到多站点文件仍然存在。
你认为我们应该覆盖多个站点还是跳过?
...
之后-网址:https://d.pr/i/5qfYhh

感谢您的测试报告,@robinwpdeveloper!混合*.orig|*.rej第二个屏幕截图中的文件是来自没有清晰应用的补丁的工件,不是构建过程本身问题的结果(请参见其他信息在里面注释:10). 清理/构建任务并不特定于单站点或多站点使用。

#19 @阿佐兹
15个月以前

补充https://github.com/WordPress/WordPress-develop/pull/4038这是基于https://github.com/WordPress/WordPress-develop/pull/3843.

在那里构建时,它还将从/build中删除CSS和JS构建的文件,以便当前行为不会更改。

这张票是在松弛(Slack)costdev的in#core。查看日志.


15个月以前

这张票是在松弛(Slack)costdev的in#core。查看日志.


14个月以前

#22 @彼得威尔森立方厘米
14个月以前

  • 关键词 请求的更改补充;犯罪远离的

@ironprogrammer在PR上添加了一些更改注释,相应地更新了此票据上的关键字,以保持里程碑报告的准确性。

@阿佐兹对发表了评论采购订单号4038:


14个月以前
#23

坦白地说,我仍然在这种方法和https://github.com/WordPress/WordPress-develop/pull/3843.

理论上最好是从/型钢混凝土批量复制之前/建造,否则应排除生成的文件。除了保持清理工作与现在一样,还应该从工作_DIR建造时。不确定这有多重要,但可能“保存起来很好”。

那么也许正确的方法是清除:文件清除/build中的所有内容(就像现在一样)以及/src中的所有构建文件。这应确保每一个新的WP构建都从全新的开始。

这张票是在松弛(Slack)costdev的in#core。查看日志.


14个月以前

这张票是在松弛(Slack)mukeshpanchal27的#core中。查看日志.


14个月以前

@铁编程器对发表了评论采购订单号4038:


14个月以前
#26

谢谢,@azaozz!`清除:文件`update,仍然存在未跟踪的文件src/wp-includes/js/tinymce/tinymce.js.它悄悄地过去了清洁的因为它被复制到源/稍后,在供应商:js.

此文件在供应商:js任务。原始补丁,47749.1差异(第1408行),通过调用清除:js在结束时再次建造(相同的结转至公关3843).

这里也应该考虑这一点吗?

@阿佐兹对发表了评论公共关系编号4038:


14个月以前
#27

仍然存在未跟踪的文件src/wp-includes/js/tinymce/tinymce.js

捕捉效果好:)
似乎在将所有js文件移动到/src/js(我认为是在WP5.0中)之后,对这个问题的清理从未得到解决。作为tinymce.js格式只有在/src中才需要,似乎只有在那里构建时才应该复制它。

#28 @铁编程器
14个月以前

  • 关键词 请求的更改远离的

谢谢你的补丁更新,@azaozz!LGTM公司👍🏻 正在删除请求的更改.

测试报告

补丁测试:https://github.com/WordPress/WordPress-develop/pull/4038

环境

  • 硬件:MacBook Pro Apple M1 Pro
  • 操作系统:macOS 12.6.3
  • 服务器:nginx/1.23.3
  • 菲律宾比索:7.4.33
  • 文字出版社:6.2-beta1-55292-src

实际结果

清洁的任务

  • 所有生成的文件都已从中删除型钢混凝土/.
  • 所有生成的文件都已从中删除建造/.

建造任务

  • 所有生成的文件都已从中删除型钢混凝土/.

构建:开发任务--自测试以来06立方英尺7

  • 未缩小tinymce.js格式保留在中型钢混凝土/.

#29 @阿佐兹
14个月以前

  • 所有者设置为阿佐兹
  • 分辨率设置为固定的
  • 状态已从更改新的关闭

55484:

生成/测试工具:运行时删除所有以前生成的文件清除:文件.

从/src中清除旧的JS、CSS和Webpack文件,以便在运行时不会自动复制到/build咕噜声建造。修复了将过时的文件和/或目录复制到/build可能导致的错误。

道具:desrosj、isabel_brison、SergeyBiryukov、铁程序员、mukesh27、robinwpdeveloper、razze007、costdev、peterwilsoncc、azaozz。

修复:#47749.

这张票是在松弛(Slack)谢尔盖的《核心》。查看日志.


14个月以前

注:请参见TracTickets公司有关使用的帮助门票。