Skip to content

Add contributions templates #996

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 8, 2016
Merged

Add contributions templates #996

merged 1 commit into from
Dec 8, 2016

Conversation

posva
Copy link
Member

@posva posva commented Dec 7, 2016

@fnlctrl I don't know what the Chinese text says 😛 I just took the vue issue template and replaced the jsfiddle links. I still think that the issue template is too long and I fear people will just ignore it and we end up not gaining any time.

@fnlctrl
Copy link
Member

fnlctrl commented Dec 7, 2016

Yeah, maybe we should make it shorter..

About the Chinese text (It's actually concise and straight to the point 😂 , though not very friendly):

中文用户请注意:
Attention Chinese users:

1. issue 只接受带重现的 bug 报告,请不要用来提问题!不符合要求的 issue 会被直接关闭。
The issue list only accepts bug reports with reproductions, please do not ask questions here! Issues that fails to comply will be immediately closed.

2. 请尽量用英文描述你的 issue,这样能够让尽可能多的人帮到你。
Please try to describe your issue in English, so that more people can help.

@posva
Copy link
Member Author

posva commented Dec 7, 2016

I'll think of something this afternoon. All of the points are important imo but the real deal is to prevent useless issues to appear and in order to do that we have to find the right balance so people really take the time to read the template

@posva
Copy link
Member Author

posva commented Dec 8, 2016

@fnlctrl After thinking about it, I don't think it's necessary to shrink the message. It's fine like this. We can start with this anyway and improve progressively. Feel free to merge

@fnlctrl fnlctrl merged commit af78335 into dev Dec 8, 2016
@fnlctrl fnlctrl deleted the guidelines branch December 8, 2016 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants