Skip to content

Commit

Permalink
update contributing guide
Browse files Browse the repository at this point in the history
  • Loading branch information
Leopoldthecoder committed Sep 19, 2016
1 parent 2921498 commit 36388de
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 12 deletions.
14 changes: 7 additions & 7 deletions .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
# Element Contributing Guide

Hi! 首先感谢你使用 Element UI。这是一个为开发者、设计师和产品经理准备的一套能让你快速搭建页面的基于 Vue 2.0 的组件库,同时我们还提供了相关设计资源供设计师使用。如果你考虑为 Element 贡献代码的话,请确保阅读下面内容
Hi! 首先感谢你使用 Element UI。这是为开发者、设计师和产品经理准备的一套能让你快速搭建页面的基于 Vue 2.0 的组件库,同时我们还提供了相关设计资源供设计师使用。如果你考虑为 Element 贡献代码,请阅读下面内容

## Issue 规范
- issue 仅仅用来提交 Bug 或 Feature 以及设计相关的内容。如果只是简单的问题,请到 Slack 或 [Gitter](https://gitter.im/ElemeFE/element) 里咨询,并且我们会直接关掉
- issue 仅仅用来提交 Bug 或 Feature 以及设计相关的内容。如果只是简单的问题,请到 Slack 或 [Gitter](https://gitter.im/ElemeFE/element) 里咨询,相关 issue 会被直接关掉

- 提交 issue 前请搜索下是否有同样的问题已经被 fix 或者回复过了。
- 提交 issue 前请搜索是否已经有同样的问题被 fix 或者回复过了。

- 请确保指明了 Element、Vue 的版本号,以及操作系统和浏览器的版本,最好能提供在线演示的链接。

Expand All @@ -18,11 +18,11 @@ Hi! 首先感谢你使用 Element UI。这是一个为开发者、设计师和

- 执行 `npm run dist` 后可以正确打包文件。

- 为了兼容和最终打包的文件体积考虑,我们的 babel 只引入了 `preset-2015`,所以不建议使用 ES2015 的 API,例如 `Array.prototype.find``Object.assign`等。如果有需要,请引入第三方的 polyfill。
- 为了兼容性以及最终打包的文件体积考虑,我们的 babel 只引入了 `preset-2015`,所以不建议使用 ES2015 的 API,例如 `Array.prototype.find``Object.assign`等。如果有需要,请引入第三方的 polyfill。

- 如果是修复 bug,请给出描述信息在 PR
- 如果是修复 bug,请在 PR 中给出描述信息

- 合并代码需要两名维护人员参与:一人进行 review 后留言 lgtm,另一人再次 review,通过后即可合并。
- 合并代码需要两名维护人员参与:一人进行 review 后 approve,另一人再次 review,通过后即可合并。

## 开发环境搭建
首先你需要 Node.js 4+ 和 NPM 3+
Expand All @@ -48,4 +48,4 @@ npm run dist
```

## 代码规范
遵循 lint 即可
遵循饿了么前端的 [ESLint](https://github.com/ElemeFE/eslint-config-elemefe) 即可
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
Thank you for contributing! Please carefully read the following before opening your issue.
-->

### Element version
### ElementUI version
<!-- 1.0.0-rc.2 -->

### OS/Browers version
Expand Down
7 changes: 4 additions & 3 deletions examples/pages/component.vue
Original file line number Diff line number Diff line change
Expand Up @@ -23,14 +23,15 @@
}
th {
text-align: left;
border-top: 1px solid #eaeefb;
background-color: #EFF2F7;
}
td, th {
border-bottom: 1px solid #eaeefb;
padding: 10px 0;
}
th {
border-top: 1px solid #eaeefb;
background-color: #EFF2F7;
th:first-child, td:first-child {
padding-left: 10px;
}
}
}
Expand Down
1 change: 0 additions & 1 deletion examples/pages/guide.vue
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
.content {
padding-left: 25px;
border-left: 1px solid #eaeefa;
margin-left: -1px;
h3 {
Expand Down

0 comments on commit 36388de

Please sign in to comment.