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

Added port & release to PR templates #152

Merged
merged 4 commits into from
Feb 12, 2021
Merged

Conversation

jennuine
Copy link
Contributor

No description provided.

@jennuine jennuine requested a review from chapulina February 12, 2021 17:03
@jennuine jennuine requested a review from j-rivero as a code owner February 12, 2021 17:03
@github-actions github-actions bot added 🏢 edifice Ignition Edifice 🏰 citadel Ignition Citadel 📜 blueprint Ignition Blueprint 🔮 dome Ignition Dome labels Feb 12, 2021
Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Contributor

@adlarkin adlarkin left a comment

Choose a reason for hiding this comment

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

Looks pretty good to me! The one thing I have noticed is that the formatting is a little odd on some of the PRs I have seen using the current templates (a few things in the checklist are scattered across multiple lines). I've attached an image below to show an example (the last three items are scattered across multiple lines). I think that we can fix this by making all of the text for every checklist item be on the same line (I left a comment below that helps better illustrate an example).

multiple lines template

.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
Signed-off-by: Jenn Nguyen <[email protected]>
@jennuine jennuine merged commit b58ce50 into ign-cmake2 Feb 12, 2021
@jennuine jennuine deleted the jennuine/templates branch February 12, 2021 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 blueprint Ignition Blueprint 🏰 citadel Ignition Citadel 🔮 dome Ignition Dome 🏢 edifice Ignition Edifice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants