跳到内容
新问题

对这个项目有疑问吗?注册一个免费的GitHub帐户以打开一个问题,并联系其维护者和社区。

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

已经在GitHub上了?登录到您的帐户

功能/#636模块特定自述文件 #664

对话

10个upsimon
复制链接
贡献者

@10个upsimon 10个upsimon 评论2023年3月2日

总结

地址#636

相关技术选择

检查表

  • PR有[焦点]基础设施标签。
  • 公关有一个[类型]标签。
  • 公关有一个里程碑或无里程碑标签。

@10上行 10个upsimon补充[类型]增强功能 改进现有功能的建议 基础设施 整体性能插件基础设施的问题 无里程碑 没有定义发布里程碑的PR 创建独立插件标签2023年3月2日
复制链接
成员

@费利萨尔茨 费利萨尔茨 留下了评论

选择隐藏此评论的原因

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

@10个upsimon你能把这里的更改限制在WebP上传模块吗?其他作为独立插件部署的模块将在单独的问题中处理。

.git属性 过时的
/modules/images/dominant-color/readme.txt导出忽略
/modules/images/fetchpriority/readme.txt导出忽略
/modules/images/webp-support/readme.txt导出忽略
/modules/images/webp-uploads/readme.txt导出忽略
复制链接
成员

选择隐藏此评论的原因

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

要求规定,作为此问题的一部分,仅添加WebP Uploads的自述文件。让我们坚持下去,忽略所有其他模块。

复制链接
贡献者 作者

选择隐藏此评论的原因

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

@费利萨尔茨我现在已经将其更改为单行glob模式,基于@乔麦吉尔的反馈,以便我们能够匹配当前和未来自述.txt文件夹

/modules/**/readme.txt导出忽略

让我知道这是否合适,或者您是否希望我将其明确定义为仅限webp-uploads,并在将来选择模块。

@10个upsimon
复制链接
贡献者 作者

@费利萨尔茨我已经回复了你的评论,将此限制为webp上传不过,我确实为所有模块实现了一个catch-all全局模式匹配自述.txt文件以便导出忽略他们。

一次@穆克什班查尔27的公关#662批准并合并后,我将增强CLI命令以处理自述.txt文件夹。

复制链接
成员

@费利萨尔茨 费利萨尔茨 留下了评论

选择隐藏此评论的原因

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

@10个upsimon感谢您的更新。我认为这里有一个误解,因为独立插件实际上是独立的。它们不应作为Performance Lab插件的扩展进行品牌/营销。

现在,作为里程碑1的一部分(参见#656)它们将作为独立插件和Performance Lab插件中的模块存在于这两个位置。但一旦我们完成了里程碑2,它们将只作为独立插件存在。

与他们一起使用Performance Lab是非常好的,如果用户想了解性能团队正在进行的工作,我们仍然鼓励他们使用Performity Lab。但是,如果有人只想使用WebP,那么只需自己安装此插件就可以了。

关于依赖#662,我认为从技术上来说,我们不必等待那个公关。一旦这个公关最终确定,它可以按原样合并,因为#662将已经复制整个目录,因此自述.txt这里将包括在内。

模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
复制链接
成员

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

选择隐藏此评论的原因

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

谢谢@10个upsimon,我留下了一个小建议,请您删除自述.txtsqlite文件? 

模块/images/webp上传/readme.txt 过时的 显示已解决 隐藏已解决
@10个upsimon
复制链接
贡献者 作者

@穆克什班查尔27 @费利萨尔茨非常感谢您的反馈和评论。我相信我已经解决了所有问题,并要求重新审查PR。我确认readme.txt按预期复制了@穆克什班查尔27的最新合并#662 @费利萨尔茨

复制链接
成员

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

选择隐藏此评论的原因

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

谢谢@10个upsimon,这些变化对我来说很好。

模块/图像/webp-uploads/readme.txt 显示已解决 隐藏已解决
合著人:Mukesh Panchal<mukeshpanchal27@users.noreply.github.com>
复制链接
成员

@费利萨尔茨 费利萨尔茨 留下了评论

选择隐藏此评论的原因

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

@10个upsimon现在这很好,只有一件小事和一件稍大的事。如果你能稍微扩展一下描述,更清楚地解释一下插件的功能,那就太棒了。

如果您只喜欢该内容,我们也可以在后续活动中进行迭代,以便取消阻止此PR。让我知道您喜欢什么。

模块/images/webp上传/readme.txt 过时的 显示已解决 隐藏已解决
模块/图像/webp-uploads/readme.txt 过时的 显示已解决 隐藏已解决
mukeshpanchal27 和其他补充提交 2023年3月15日15:15
合著人:Felix Arntz<felixantz@users.noreply.github.com>
…om:WordPress/performance添加/发布-636-module-specific-readme文件
复制链接
成员

@费利萨尔茨 费利萨尔茨 留下了评论

选择隐藏此评论的原因

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

@10上行感谢您的更新,LGTM!

@费利萨尔茨 费利萨尔茨合并提交e279a6进入之内 功能/创建标准插件 2023年3月16日
@费利萨尔茨 费利萨尔茨删除了 添加/发布636-module-specific-readme文件 分支 2023年3月16日15:54
免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
标签
基础设施 整体性能插件基础设施的问题 无里程碑 没有定义发布里程碑的PR [类型]增强功能 改进现有功能的建议
项目
还没有
开发

成功合并此请求可能会解决这些问题。

3名参与者