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.

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

Improved GitHub issues layout #15677

Merged
merged 1 commit into from
Mar 12, 2021
Merged

Improved GitHub issues layout #15677

merged 1 commit into from
Mar 12, 2021

Conversation

gastaldi
Copy link
Contributor

@gastaldi gastaldi commented Mar 12, 2021 •
edited
Loading

This improves issues' readability. Example: #15679

@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Mar 12, 2021
@gastaldi gastaldi requested a review from gsmet March 12, 2021 14:52
Base automatically changed from master to main March 12, 2021 15:55
gsmet
gsmet requested changes Mar 12, 2021
View reviewed changes
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Good idea but I wouldn't remove the trailing spaces in this case.

.github/ISSUE_TEMPLATE/bug_report.md Outdated
3.
1.
2.
3.
Copy link
Member

Choose a reason for hiding this comment

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

I would have kept the spaces here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I blame IDEA for that 😝

.github/ISSUE_TEMPLATE/bug_report.md Outdated
- Output of `java -version`:
- GraalVM version (if different from Java):
- Quarkus version or git rev:
- Build tool (ie. output of `mvnw --version` or `gradlew --version`):
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

@gastaldi gastaldi requested a review from gsmet March 12, 2021 19:35
@gastaldi
Copy link
Contributor Author

I made the Environment section in the Bug Report template a bit different, using sections instead of single line

@gsmet gsmet merged commit fac33e9 into quarkusio:main Mar 12, 2021
@quarkus-bot quarkus-bot bot added this to the 1.13 - master milestone Mar 12, 2021
@gastaldi gastaldi deleted the issues branch March 12, 2021 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsmet gsmet gsmet approved these changes

Assignees
No one assigned
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure
Projects
None yet
Milestone
1.13.0.CR1
Development

Successfully merging this pull request may close these issues.
2 participants
@gastaldi @gsmet