提交者指南
这些指南适用于在 ansible 和 ansible-collections GitHub 组织中的存储库拥有提交权限的人员。
Ansible-core 的提交者必须是 Red Hat 员工,并作为 Ansible 核心团队的成员。 Ansible 集合 的提交者是社区成员或 Ansible 工程团队成员。 请在提交之前阅读这些指南。
这些指南适用于所有人。 同时,这不是一个流程文档。 所以请使用您的良好判断。 我们授予您提交访问权限,因为我们信任您的判断力。
也就是说,请明智地使用这份信任。
如果您滥用信任,破坏了组件和构建等,信任级别将会下降,您可能会被要求不要提交,或者您可能会失去提交权限。
ansible-core 的功能、高级设计和路线图
作为核心团队成员,您是开发 路线图 的团队中不可或缺的一部分。 请积极参与,并推动您希望看到的特性和修复。 同时请记住,Red Hat 作为一家公司,将承诺为各种版本提供某些特性、修复、API 等。 Red Hat 公司和 Ansible 团队必须按计划完成并发布这些更改。 对用户、社区和客户的义务必须放在首位。 由于这些承诺,您希望自己开发的功能可能不会进入某个版本,如果它会影响 Ansible 中的许多其他部分。
任何其他新功能和对高级设计的更改都应通过提案流程(待定)进行,以确保社区和核心团队有机会审查该想法并批准它。 核心团队对根据提案将新功能合并到 Ansible-core 中负有唯一责任。
Ansible 集合的功能、高级设计和路线图
集合维护者定义集合本身的功能、高级设计和路线图,并负责根据与社区讨论的提案将新功能合并到 Ansible 集合 中。
我们在 GitHub 上的工作流程
作为提交者,您可能已经知道这一点,但我们的工作流程构成了我们许多团队政策的基础。 请确保您了解以下工作流程步骤
将您想要进行工作的存储库分叉到您自己的个人存储库中
在您需要提交的特定分支上进行工作
创建回上游存储库的拉取请求,并标记您想让其审查的人;为您的拉取请求指定某人为主要“所有者”
根据提供的评论调整代码
请存储库提交者中的某人进行最终审查并合并
提交者工作流程补充
核心团队知道这有时可能是一个困难的过程。 有时,团队会通过进行直接提交或合并他们自己的拉取请求来违反规则。 本节是一套指南。 如果您正在更改文档中的逗号,或进行非常小的更改,您可以根据自己的判断行事。 这是另一件信任的事情。 该流程对于任何重大更改都至关重要,但对于小事情或快速完成某些事情,请根据您的最佳判断行事,并确保团队中的人员了解您的工作。
核心角色
通用规则
拥有直接提交访问权限的个人被赋予了可以让他们做各种事情的权限——可能比我们能写下来的还要多。 与其说是规则,不如说是通用指南,拥有此权限的个人应使用其最佳判断力。
不要
直接提交。
合并您自己的拉取请求。 其他人应该有机会审查并批准拉取请求的合并。 如果您是核心提交者,对于非常小的更改,您在这里有一点回旋余地。
忘记替代环境。 考虑替代方案——是的,人们有糟糕的环境,但他们是最需要我们帮助的人。
拖累您的社区团队成员。 讨论您审查的任何拉取请求的技术优点。 避免消极和人身攻击。 有关成为优秀社区成员的更多指导,请阅读我们的 社区行为准则。
忘记维护负担。 高维护成本的功能可能不值得添加。
破坏 playbook。 始终牢记向后兼容性。
忘记保持简单。 复杂性会导致各种问题。
做
压缩提交,尽可能避免合并,如有必要,使用 GitHub 的压缩提交或 cherry-pick(bisect 感谢您)。
积极参与。 在项目中没有活动(通过合并、分类、提交等)的提交者将被暂停权限。
考虑向后兼容性(回到“不要破坏现有的 playbook”)。
编写 测试,并确保您正在审查的其他人的拉取请求得到了很好的覆盖。 具有测试的拉取请求比没有测试但应该包含测试的拉取请求具有更高的优先级。 虽然并非所有更改都需要测试,但请务必为新功能、错误修复和功能更改添加测试。
与其他提交者讨论,尤其是在您不确定某些事情时。
编写文档! 如果您的拉取请求是新功能或行为更改,请确保您已更新所有相关文档或已通知相关人员这样做。 它还有助于添加与之兼容的
ansible-core
或collection
版本(以避免稳定版本和开发版本文档之间的混淆,以及向后兼容性等)。考虑范围,有时可以将修复程序泛化。
保持简单,这样才能保持可维护性、可调试性和可理解性。
提交者应继续遵循 Ansible 社区其他成员遵循的相同社区和贡献指南。