使WordPress成为核心

开的6个月前

上次修改时间6周前

#60129 重新打开的 任务(幸运的)

不包括Trac票证时对PR进行评论

报告人: desrosj剖面 德斯罗什 所有者: 德斯罗什的个人资料 德斯罗什
里程碑: 6.6 优先: 正常的
严重程度: 正常的 版本:
组件: 构建/测试工具 关键词: 好的第一只虫子 has-patch接口 犯罪
重点: 复写的副本:

描述

在GitHub上使用拉请求向Core贡献时需要有Trac票才能考虑供款.

尽管拉取请求模板注意到了这一点,但很容易忘记或忽略。

在Pull Request Comments工作流中添加作业,以便在缺少Trac票证链接时添加提醒注释,这是一种简单的方法,有助于避免捐款在洗牌过程中丢失。

更改历史记录(16)

这张票是在采购订单号6060WordPress/WordPress-develop开发通过@阿那马里亚帕奇.


4个月以前
#1

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

在GitHub上使用拉请求向Core贡献时需要Trac票才能考虑供款.

尽管pull请求模板注意到了这一点,但很容易忘记或忽略。

在Pull Request Comments工作流中添加作业,以便在缺少Trac票证链接时添加提醒注释,这是一种简单的方法,有助于避免捐款在洗牌过程中丢失。

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

@德斯罗什对发表了评论采购订单号6060:


4个月以前
#3

今天我想得更多了。我认为我们还应该尝试检查是否存在之前的评论,以避免在PR上对同一事物进行多次评论。

@阿那马里亚帕奇对发表了评论采购订单号6060:


4个月以前
#4

感谢您的评论@desrosj和@peterwilsoncc!

我会在接下来的几天有空的时候进行所要求的更改。如果您有任何进一步的建议,请随时告诉我!

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


4个月以前

#7 @瑞士风格
4个月以前

  • 类型已从更改缺陷(bug)任务(幸运的)

#8 @瑞士风格
4个月以前

  • 里程碑已从更改6.56.6

#9 @devsahadat公司
3个月以前

当通过GitHub拉取请求向WordPress Core贡献内容时,记住包含Trac票证链接是至关重要的。尽管模板中有提醒,但还是会发生疏忽。
在PR Comments工作流中执行作业,提示缺少Trac票证链接,可以防止捐款流失。此外,考虑@anamarijapapic关于避免对公关重复评论的建议,将进一步简化流程。

#10个 @阿那马里亚帕奇
3个月以前

@desrosj@peterwilsoncc@swissspidy有什么更新吗?

#11 @德斯罗什
7周以前

  • 关键词 犯罪补充
  • 所有者设置为德斯罗什
  • 状态已从更改新的认可的

嗨@anamarijapapic!感谢您为此所做的工作,很抱歉花了这么长时间才回过头来!

我认为这看起来状态不错。让我们试试看!

#12 @德斯罗什
7周以前

  • 分辨率设置为固定的
  • 状态已从更改认可的关闭

58092:

构建/测试工具:提醒参与者包含Trac票证链接。

使用为WordPress撰稿wordpress开发在GitHub上协作、测试和审查对代码库的建议更改是一种有用的方式。然而,所需的条件之一是包含指向相应Trac票证的链接。这确保了公关和相关活动被列在Trac票据上,作为真相的来源。

很容易忘记这一点,新的贡献者并不总是意识到这一要求。这将添加GitHub Actions作业,该作业将在不包含Trac票证时添加注释作为提醒。

等待真的结束了吗?两千年了。

Props anamarijapapic,彼得威尔森公司。
修复#60129.

#14 @海洋90
7周以前

添加链接后是否应删除评论?

#15 @德斯罗什
7周以前

  • 分辨率 固定的删除
  • 状态已从更改关闭重新打开的

@ocean90限制机器人在PR上的评论数量可能是个好主意。

看起来有一个已编辑的活动类型拉动请求目标/拉_请求可以用来做这个。

#16 @德斯罗什
6周以前

  • 分辨率设置为固定的
  • 状态已从更改重新打开的关闭

58120:

生成/测试工具:删除检查最新的输入设置-节点.

这个检查最新设置用于确保在GitHub Action工作流中始终安装最新版本的Node.js。

此输入已在中添加并打开[57212]当Node.js的最低要求版本被提升到20.10.0。由于GitHub Action运行程序映像更新是在几天内滚动部署的,因此满足此新要求的Node.js版本并不总是存在(尤其是在Windows运行程序上)。使用此输入是一个临时修复,以确保Core测试工作流的稳定性。

这个检查最新输入确实有一些副作用。两个例子是:

  • 每次都会执行一个附加请求来检查最新版本设置-节点在启用此选项的情况下使用。当Node.js的更新版本可用时,会有更多的请求下载并安装该版本。
  • 当Node.js的新版本发布时,核心工作流会立即切换到新版本,这可能会有未发现的错误或回归。

由于Windows上Node.js 20.13.0中的回归,后者今天出现了(请参阅https://github.com/nodejs/node/issues/52884).

一段时间过去了,现在所有GitHub Action运行程序都可以可靠地使用>=20.10.0的版本。运行最新版本Node.js对于Core的测试设置也不重要,因此检查版本可以安全移除以解决上述两种副作用。

道具约十亿。
修复#60129.

#17 @约十亿
6周以前

  • 分辨率 固定的删除
  • 状态已从更改关闭重新打开的
注:请参见TracTickets公司有关使用的帮助门票。