/video.js

Add ISSUE_TEMPLATE for github. #3117

Closed
wants to merge 5 commits into from
+48 −0

Labels

confirmed patch

Milestone

No milestone

Assignees

No one assigned

3 participants

@gkatsev @dmlap @heff
@gkatsev
Video.js member
gkatsev commented Feb 17, 2016

See https://github.com/blog/2111-issue-and-pull-request-templates

@dmlap
Video.js member
dmlap commented Feb 17, 2016

Cool! Notes:

  • link to jsbin.com/axedog for the reduced test cases?
  • The STR numbering doesn't render correctly in github markdown right now
@gkatsev
Video.js member
gkatsev commented Feb 17, 2016

Yeah, the number doesn't render correctly without content. When new issues are included, they're in "write" mode and not "preview" mode, so, it should be OK.

@dmlap
Video.js member
dmlap commented Feb 17, 2016

The PR template is awesome for new features. I wouldn't want to set the bar that high for simple bug fixes, though. Any way to split the difference?

@gkatsev
Video.js member
gkatsev commented Feb 17, 2016

Not sure. A lot of that applies should apply to "simple bug fixes" as well. Maybe we just need better/simpler wording.

@heff
Video.js member
heff commented Feb 17, 2016

Nice! Love this feature. lgtm

@gkatsev
Video.js member
gkatsev commented Feb 18, 2016

@dmlap I updated the wording in the PR Template. Please take another look.

@dmlap
Video.js member
dmlap commented Feb 18, 2016

👍

@gkatsev gkatsev added confirmed and removed needs: LGTM labels Feb 18, 2016
@gkatsev gkatsev added a commit that closed this pull request Feb 18, 2016
@gkatsev gkatsev @gkatsev added issue and PR templates for github. closes #3117 050f23d
@gkatsev gkatsev closed this in 050f23d Feb 18, 2016
@gkatsev gkatsev deleted the gkatsev:issue-templates branch Feb 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment