跳到内容
新问题

有关于这个项目的问题吗？注册一个免费的GitHub帐户以打开一个问题，并联系其维护者和社区。

注册GitHub

单击“注册GitHub”，表示您同意我们的服务条款和隐私声明。我们偶尔会向您发送与帐户相关的电子邮件。

已经在GitHub上了？登录到您的帐户
跳到底

添加自动性能度量工作流 #4139

关闭
想要合并107提交到从
关闭

添加自动性能度量工作流 #4139

想要合并107提交到从

对话

乔麦吉尔
复制链接
成员

Trac票：https://core.trac.wordpress.org/tickt/57687

此Pull请求仅用于代码审查。请将所有其他讨论保留在Trac记录单中。不要合并此Pull请求。请参见GitHub拉取代码审查请求有关更多详细信息，请参阅《核心手册》。

@穆克什班查尔27
添加性能工作流
c334215号
@穆克什班查尔27
在文件末尾添加了换行符
第26天第8页94
@穆克什班查尔27
处理审查反馈
5455b52号
@穆克什班查尔27
更新工作流步骤
53亿445天
@穆克什班查尔27
为服务器计时API添加MU插件
ba128a1型
@穆克什班查尔27
将分支“feature/automated-performance-testing-mvp”合并到add/ser…
a061f65 
…ver-timing-api-mu插件
@穆克什班查尔27
添加配置环境变量步骤
2bd139b号
@穆克什班查尔27
处理审查反馈
b2581a7型
@费利萨尔茨
合并拉请求 #6 从10开始/setup-initial-performance-action
015383页 
为性能测试设置初始GitHub操作
@mukeshpanchal27
处理审查反馈
ba1a5fc公司
@穆克什班查尔27
将分支“feature/automated-performance-testing-mvp”合并到add/ser…
09616a5号 
…ver-timing-api-mu插件
@穆克什班查尔27
将分支“WordPress:trunk”合并为feature/automated-performance-tes…
0a3bddf 
…ting-mvp（音）
@mukeshpanchal27
将分支“WordPress:trunk”合并为feature/automated-performance-tes…
b8ff136 
…ting-mvp（音）
@穆克什班查尔27
更新MU插件路径
85a5c96号
@穆克什班查尔27
合并拉请求 #7 从10up/add/server定时api mu插件
数据库206a3 
服务器定时API的MU插件
@穆克什班查尔27
将分支“WordPress:trunk”合并为feature/automated-performance-tes…
0c002a2号机组 
…ting-mvp（音）
@穆克什班查尔27
初始更改
公元1142d
@穆克什班查尔27
初始变更
4c655fd型
@穆克什班查尔27
添加缺少的文件
483天780
@穆克什班查尔27
将分支“feature/automated-performance-testing-mvp”合并到add/ser…
2立方英尺8 
…ver-timing-metrics-gh-工作流
@乔麦吉尔
恢复以前的性能工作流步骤
20年5月5日
@乔麦吉尔
更新性能测试
00c6d45 
这会将使用`wp-scripts`中的`test-e2e`命令的性能测试移动到它们自己的目录中，并更新规范以修复以前触发的一些JS错误。
@穆克什班查尔27
合并拉请求 #21 从10up/add/server-timing-fix
3b130巴 
性能工作流修复
@穆克什班查尔27
修复合并后的错误
7e1b9ba
@穆克什班查尔27
其他更改
4a692e5标准
@乔麦吉尔
还原性能工作流更改
43f171d页
@乔麦吉尔
忽略teset结果文件
ae71d89型
@乔麦吉尔
删除不必要的包.json更改
1984年3月
@乔麦吉尔
在工作流期间运行性能测试
1749ea2个
@乔麦吉尔
从生成目录运行工作流
335493a个
德斯罗什
德斯罗什 检验过的 2023年3月2日
查看已审阅的更改
.github/工作流/性能.yml 过时的 显示已解决 隐藏已解决
德斯罗什
德斯罗什 检验过的 2023年3月2日
查看已审阅的更改
.github/工作流/性能.yml 过时的 显示已解决 隐藏已解决
@乔麦吉尔
通过env变量设置基本版本
04c0e39号 
这将添加一个新的BASE_HASH环境变量，该变量在我们引用基本版本或哈希的整个工作流中使用。
成本开发
成本开发 批准了这些更改 2023年3月3日
查看已审阅的更改
复制链接
贡献者

@成本开发 成本开发 留下了评论 •
已编辑
加载

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

感谢ping审阅此公关@穆克什班查尔27!

除了一个小的评论格式更改外，我没有发现任何在之前的讨论中没有提到和解决的问题。干得好！

felixarntz公司
felixarntz公司 批准了这些更改 2023年3月3日
查看已审阅的更改
复制链接
成员

@费利萨尔茨 felixarntz公司 留下了评论

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

@乔麦吉尔 @穆克什班查尔27干得好，LGTM！🚢

这是一个很好的起点。合并后，让我们评估我们得到的指标的方差，看看它们的可靠性。有几个后续问题需要进一步研究，例如其他指标（对于客户端来说最重要的是LCP，也可能是一般的Web Vitals）或其他场景（可能是WP的翻译版本，带有对象缓存的站点，诸如此类的想法）。当然，我们还必须记住保持工作流的持续时间可管理，因此需要谨慎做出这些决定。

德斯罗什
德斯罗什 检验过的 2023年3月3日
查看已审阅的更改
.github/工作流/性能.yml
-name:打印基线性能测试结果
运行：“node./tests/performance/results.js--prefix=base”

-name:设置基础sha
复制链接
贡献者

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

建议的更改
-名称:设置底座沙
-名称:设置底座SHA公司

缩写。

德斯罗什
德斯罗什 批准了这些更改 2023年3月3日
查看已审阅的更改
复制链接
贡献者

@德斯罗什 德斯罗什 留下了评论

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

我对所有的修改都很满意。非常感谢大家！真的很期待合并此内容，这样我们就可以开始跟踪性能更改对更改集的更改。

@乔麦吉尔
复制链接
成员 作者

在中提交55459

@乔麦吉尔 乔麦吉尔删除了 功能/自动性能测试-mvp 分支 2023年6月9日17:18
免费注册 在GitHub上加入此对话.已经有帐户了吗？登录以发表评论
审阅者

@你知道riad 你知道riad 你知道riad留下了评论

@瑞士风格 瑞士风格 瑞士人留下评论

@穆克什班查尔27 穆克什班查尔27 mukeshpanchal27留下评论

@德斯罗什 德斯罗什 desrosj批准了这些更改

@费利萨尔茨 felixarntz公司 费利萨尔茨批准了这些修改

@成本开发 成本开发 costdev批准了这些更改

@亚当西尔弗斯坦 亚当西尔弗斯坦 等待亚当西尔弗斯坦的审查请求

受让人

@乔麦吉尔 乔麦吉尔

@穆克什班查尔27 穆克什班查尔27

标签
还没有
项目
还没有
里程碑
无里程碑
开发

成功合并此请求可能会解决这些问题。
7名参与者
@乔麦吉尔 @你知道riad @德斯罗什 @瑞士风格 @费利萨尔茨 @穆克什班查尔27 @成本开发