Skip to content

ci: Simplify GH bug report template#3381

Merged
polarathene merged 2 commits intomasterfrom
refactor/gh-template-bugreport-simplify
Jun 1, 2023
Merged

ci: Simplify GH bug report template#3381
polarathene merged 2 commits intomasterfrom
refactor/gh-template-bugreport-simplify

Conversation

@polarathene
Copy link
Copy Markdown
Member

Description

Simplify the bug report form further by dropping / merging form sections. Partially inspired uptime-kuma report.


Presently the bug report form seems to have plenty of redundant content that maintainers don't leverage as context for responding or triaging any differently. They were originally intended to save us time but I don't recall that happening, sometimes information by the reporter was still communicated at a later stage.

In my experience, it's just additional noise for us and minor friction to the reporter. When the debug docs page is improved, we can probably simplify the "Preliminary Checks" section too (or migrate common gotchas to still be visible on the bug report page as markdown comment placeholder in the "What Happened?" section).

Changes by Commits:

    • Minor revisions and formatting changes (multi-line pipe operator, emoji, fix typos, etc).
    • Collapsed OS + Arch into single input field (not much benefit from the two additional dropdown items).
    • Description/reproduction and expectation sections revised (expectation intent is typically inferred by the issue description, while detailed reproduction steps can belong a separate optional section).
    • Removed platform dropdown (Windows and macOS are mentioned in description as unsupported).
    • Removed experience checkboxes (context doesn't really change responses).
    • Removed the orchestrator dropdown (we don't seem to use this information, it's just noise)
    • Relocate the DMS version + OS/Arch sections to come after the Reproduction steps.

Simplify the bug report form further by dropping / merging form sections.

- Minor revisions and formatting changes.
- Collapsed OS + Arch into single input field.
- Description/reproduction and expectation sections revised. Expectation intent should be covered/inferred by the issue description, while detailed reproduction steps can be it's own optional section.
- Removed platform dropdown (Windows and macOS are mentioned in description as unsupported).
- Removed experience checkboxes (context doesn't really change responses).
- Remove the orchestrator dropdown (we don't seem to use this information, it's just noise)
- Relocate the DMS version + OS/Arch sections to come after the Reproduction steps.
@polarathene polarathene added area/ci kind/improvement Improve an existing feature, configuration file or the documentation labels May 31, 2023
@polarathene polarathene added this to the v13.0.0 milestone May 31, 2023
@polarathene polarathene self-assigned this May 31, 2023
- the [documentation on debugging](https://docker-mailserver.github.io/docker-mailserver/latest/config/debugging/), tried the proposed steps to debug the problem, but was still unable to resolve the issue;
- this project's [Code of Conduct](https://github.com/docker-mailserver/docker-mailserver/blob/master/CODE_OF_CONDUCT.md) and I agree;
- the [documentation about filing a bug report](https://docker-mailserver.github.io/docker-mailserver/latest/contributing/issues-and-pull-requests/#filing-a-bug-report).
I have read:
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I intentionally added two spaces at the end of this line, thinking it might help with the initial formatting.

When filing the issue, it's not properly markdown formatted into multi-line list, only after submission. Seems to be due to a restriction on label perhaps? Whereas if it was in description it'd be rendered as a list.

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.

I intentionally added two spaces at the end of this line, thinking it might help with the initial formatting.

I noticed this behavior as well. Does your fix with the spaces improve the situation? The preview is still a bit gobbled I think..

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Does your fix with the spaces improve the situation?

🤷‍♂️ I haven't tried it out on a test repo sorry 😅

A future revision can move this into description, with the first two checkbox items being added to debug docs.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Merged, it did not help 😢

Form looks better overall though I think 😎

Copy link
Copy Markdown
Member

@georglauterbach georglauterbach left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏼

@polarathene polarathene merged commit e680622 into master Jun 1, 2023
@polarathene polarathene deleted the refactor/gh-template-bugreport-simplify branch June 1, 2023 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci kind/improvement Improve an existing feature, configuration file or the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants