代码贡献指南

很高兴你对Matermost open source project感兴趣。为了帮助翻译,请参阅本地化过程。 对于代码贡献,这里是过程:

1 选择一个Ticket

  1. Help Wanted GitHub问题列表中选择一个ticket。
  2. 在开始工作之前,comment让人们知道你正在处理这个问题。
  3. 如果您有任何问题,请将其发布到Mattermost forum或加入Contributors频道,并宣布您想要处理的故障单,以便将其分配给您。

欢迎没有ticket的轻微更正和改善的Pull requests

如果对现有行为的改动较小,少于20行的代码更改,则可以提交一个缺陷或增量改进的PR,不需要Help Wanted ticket。 所有提交的没有相应标签的请求将首先由核心团队产品经理审核。 一些小的更正和改进的例子包括:

较大的pull request,包括需要修改文档或用户期望的请求,需要核心团队打开Help Wanted,以便测试,记录和支持更改。
与核心团队讨论opening a Help Wanted ticket的最佳方法是在the feature idea forum中开始对话

2 安装Mattermost并设置你的Forks

一旦你有ticket:

  1. 按照developer setup instructions说明安装Mattermost。
  2. 在您的mattermost-server或mattermost-webapp repository中创建一个分支,并使用以下命令将<branch name>设置为正在使用的ticket的ID,例如PLT-394git checkout -b <branch name>
  3. 看看developer flow,了解如何使用Mattermost代码库。

3 准备Pull Request

在提交poll request(PR)之前,请检查:

  • 您已经签署了Contributor License Agreement,因此您可以将其添加到Mattermost Approved Contributor List中。
  • 你有一个用于改善代码的Help Wanted ticket
  • 你的代码遵循Mattermost Style Guide,你已经运行了make check-style来检查样式错误。
  • 包括新的服务器端功能的单元测试。
  • 用户界面中的字符串包含在i18n/en.json服务器和webapp i18n/en.json本地化文件中。其他语言的文件将通过Mattermost Transtation Server自动更新,不需要包含在拉取请求中。
  • 变化符合Fast, Obvious, Forgiving的用户体验指南。
  • 如果更改要求用户理解新的概念或做出决定,PR帮助文档将提交给mattermost/docs
  • 改变经过彻底的测试。如果您的更改涉及到文本处理,请确保在提交PR之前至少在/tests中运行降级测试。运行loadtests:
    • 转到System Console > Developer并将Enable Testing Commands设置为true
    • 运行/test url test-markdown-basics.md并按照说明操作
    • 运行/test url test-markdown-lists.md并按照说明操作
    • 运行/test url test-tables.md并按照说明操作
  • 确认你squashed your commits

4 提交Pull Request

提交PR时,请检查:

  1. PR is submitted against master
  2. PR标题以Jira Ticket ID开头(例如PLT-394:见examples
  3. PR comment描述了这些变化以及该功能应该如何工作
  4. 多个存储库中的多个相关PR必须在说明中包含它们之间的依赖关系。

5 管理一个Open Pull Request

提交PR后,合并之前:

  1. 自动构建过程必须通过
    • 如果构建失败,请检查错误日志以缩小原因。
    • 有时,由于Travis CI中的问题,多个构建测试中的一个将会随机失败。 如果您只看到一个构建失败并且没有明确的错误消息,则可能是一个随机问题。 添加评论,以便更改的审阅者可以重新运行构建,或者关闭公共关系并重新提交,通常会清除问题。
  2. PM审查必须通过
    • 产品经理将检查pull request以确保:
    • 产品经理可能会回来一些错误或用户界面的改进来解决之前拉动请求移动开发审查。
  3. CSS审查必须通过
    • 任何包含CSS更改的pull request都应该由UX和HTML/CSS开发人员进行审查。
  4. Dev审查必须通过
  • 两个核心提交者将审查拉取请求,并提供反馈或批准公关。
  • 在拉取请求准备合并之前,需要解决任何注释。

如果您的邮寄地址包含在注册的Contributor License Agreement中,则在您的第一个拉取请求合并后,您可能会收到一个限量版Mattermost杯子作为感谢礼物。

核心提交者

Mattermost存储库上的核心提交者由经过审核的核心团队成员组成,包括社区贡献者以及来自Mattermost,Inc.的员工,这些人员都值得信赖来审查和合并PR。

  • 核心提交者包括:Coreyhulen,crspeller,csduarte,enahum,grundleborg,hmhealey,jwilander
  • 产品经理包括:asaadmahmood,esethna,it33,jasonblais,lfbrock
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 217,406评论 6 503
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 92,732评论 3 393
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 163,711评论 0 353
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 58,380评论 1 293
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 67,432评论 6 392
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 51,301评论 1 301
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 40,145评论 3 418
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 39,008评论 0 276
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 45,443评论 1 314
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,649评论 3 334
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,795评论 1 347
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,501评论 5 345
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 41,119评论 3 328
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,731评论 0 22
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,865评论 1 269
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,899评论 2 370
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,724评论 2 354

推荐阅读更多精彩内容

  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,654评论 18 139
  • 多种多样的工作流使得在项目中实施Git时变得难以选择。这份教程提供了一个出发点,调查企业团队最常见的Git工作流。...
    JSErik阅读 4,403评论 2 8
  • 宇彤老师第三期开营了:通过昨晚的学习,开始反思自已的发音、吸气、呼气.......开始慢慢体会美丽智慧的宇彤老师"...
    云淡风轻一阅读 198评论 0 0
  • 随着公司团队的不断扩张与年轻化,大多数管理者会遇到这样一个问题——如何管理 90 后员工? 许多人给 90 后打上...
    东莱山人阅读 520评论 0 2
  • 这几天不知道咋回事看同事特别不顺眼,对待工作极其不负责任,总是喜欢拖,一直到不得不做了才火急火燎的去完成。...
    我心翱翔阅读 199评论 0 1