Skip to content

Update ISSUE_TEMPLATE with clearer instructions #3113

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

Closed
wants to merge 2 commits into from
Closed

Update ISSUE_TEMPLATE with clearer instructions #3113

wants to merge 2 commits into from

Conversation

jchannon
Copy link
Contributor

@jchannon jchannon commented May 3, 2018

I went to Antiforgery repo, it said raise an issue here. I read this template and it told me to create it in a specific repository. Um, what? This should hopefully make it clearer.

I went to Antiforgery repo, it said raise an issue here. I read this template and it told me to create it in a specific repository. Um, what?  This should hopefully make it clearer.
@Tratcher
Copy link
Member

Tratcher commented May 8, 2018

This is already stated on line 15 https://github.com/aspnet/Home/pull/3113/files#diff-1a1c3dd142f76a5fad803a0c52839881L15. At least remove the duplication?

@jchannon
Copy link
Contributor Author

jchannon commented May 8, 2018

Removed the duplication, I think that was the line that was confusing 😄

@muratg
Copy link
Contributor

muratg commented May 11, 2018

cc @Eilon

@Tratcher Tratcher mentioned this pull request May 15, 2018
@natemcmaster natemcmaster changed the base branch from dev to master July 6, 2018 19:34
@Eilon
Copy link
Contributor

Eilon commented Jul 13, 2018

Hi @jchannon thanks for the PR. I just updated the issue template with a bunch of info, including a clarification on repos: https://github.com/aspnet/Home/blob/master/.github/ISSUE_TEMPLATE.md

Thanks,
Eilon

(BTW sorry for the delay - I was on paternity leave + vacation for a while!)

@Eilon Eilon closed this Jul 13, 2018
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.

4 participants