Skip to content

Added pull request template#175

Open
sonamtenzin2 wants to merge 2 commits into
mainfrom
sonamt/pr-desc
Open

Added pull request template#175
sonamtenzin2 wants to merge 2 commits into
mainfrom
sonamt/pr-desc

Conversation

@sonamtenzin2

Copy link
Copy Markdown

No description provided.

@sonamtenzin2
sonamtenzin2 requested review from a team as code owners April 16, 2025 09:59
mukeshjc
mukeshjc previously approved these changes Apr 16, 2025
Comment thread .github/PULL_REQUEST_TEMPLATE.md Outdated
Comment on lines +10 to +14
- [JIRA](https://hashicorp.atlassian.net/browse/xxxx)
- [RFC](https://docs.google.com/document/d/xxxx)
- [PRD](https://docs.google.com/document/d/xxxx)
- [Related PR](https://github.com/hashicorp/atlas/pull/xxxx)
- [Documentation](https://github.com/hashicorp/terraform-docs-common/pull/xxxx)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there a reason for these link examples?

This is a public repository, so external contributors won't be using Jira, RFCs, or PRDs and it's likely internal contributors won't have 2 of these 3 links depending on the PR request purpose. The bottom two are "randomly selected" pointers, one of which is not a public repo. I am of the opinion these should be removed.

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