Skip to content

Update ISSUE_TEMPLATE.md [triage-skip] #13

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
May 25, 2017
Merged

Update ISSUE_TEMPLATE.md [triage-skip] #13

merged 6 commits into from
May 25, 2017

Conversation

@jshcrowthe
Copy link
Contributor

@jshcrowthe jshcrowthe commented May 18, 2017

Updates the issue template to include a link to a template JSBin that can be used to reproduce issues for us.

@jshcrowthe jshcrowthe requested a review from samtstern May 18, 2017 01:10
jshcrowthe added 3 commits May 25, 2017 12:28
@jshcrowthe
refactor(github-meta): leverage .github directory
aa29056
@jshcrowthe
docs(ISSUE_TEMPLATE): added JSBin template for repros
f707e32 
To help encourage people to reproduce issues, I've made a small JSBin that will load Firebase and
give developers a REPL environment to demonstrate the issue.
@jshcrowthe
docs(ISSUE_TEMPLATE): restructuring issue template to only display ne…
6b9bcef 
…eded info
@samtstern
Copy link
Contributor

samtstern commented May 25, 2017

  1. Using comments doesn't really matter since when they fill out the template they see the markdown and it's not even syntax highlighted.

  2. Right now the bot needs to see "### [REQUIRED]" to start a required section. That's configurable but it needs to be on a single line.

  3. I will have to update the config to specify a special location for ISSUE_TEMPLATE if you wanna move it to .github but that's not too hard if you want that.

@jshcrowthe
Copy link
Contributor Author

jshcrowthe commented May 25, 2017

Resolutions from a conversation with @samtstern:

  1. Leveraging comments is a nice way to ensure that people who are writing issues see the text, but people triaging them don't have text they don't need.
  2. Reverting the [REQUIRED] header changes.
  3. Augmenting the bot to support the .github directory

@samtstern samtstern merged commit 9d74436 into master May 25, 2017
@jshcrowthe jshcrowthe deleted the issue-template branch June 1, 2017 17:40
@firebase firebase locked and limited conversation to collaborators Oct 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@samtstern samtstern samtstern approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jshcrowthe @samtstern