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

Update Issue templates #124

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

hexadecimal233
Copy link

See #86

@Su5eD Su5eD self-assigned this Aug 27, 2023
@Su5eD Su5eD added the enhancement New feature or request label Aug 27, 2023
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
@@ -37,7 +48,7 @@ body:
label: Logs
description: |
If applicable (crash, error output in console), please provide your crash report or latest.log
To upload logs, use an external site to post the crash report, such as [Ubuntu Pastebin](https://paste.ubuntu.com/), [Pastebin](http://pastebin.com/), or [Github Gist](https://gist.github.com/)
To upload logs, use an external site to post the crash report, such as [**MCLogs (Recommended)**](https://mclo.gs/), [Ubuntu Pastebin](https://paste.ubuntu.com/), or [Pastebin](http://pastebin.com/)
Copy link
Member

Choose a reason for hiding this comment

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

Looks like paste.ubuntu.com requires registration now, and I could not find any information about its size limit. We should replace it with GitHub Gist instead, which still requires sign up but allows up to 100 MBs of data.

Suggested change
To upload logs, use an external site to post the crash report, such as [**MCLogs (Recommended)**](https://mclo.gs/), [Ubuntu Pastebin](https://paste.ubuntu.com/), or [Pastebin](http://pastebin.com/)
To upload logs, use an external site to post the crash report, such as [**MCLogs (Recommended)**](https://mclo.gs/), [Github Gist](https://gist.github.com/), or [Pastebin](http://pastebin.com/)

Copy link
Author

Choose a reason for hiding this comment

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

Can't find a reason post logs on gist (imo, not ideal). You can just upload the additional files directly into the issue.

Choose a reason for hiding this comment

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

How about we just remove paste.ubuntu.com, see no reason why we would need it tbh (same for gist). pastebin.com seems fine as an alternative, but can we use a https link instead?

Copy link
Member

Choose a reason for hiding this comment

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

Is there any reason to have separate mod malfunction and mod crash issue templates? Both cases are pretty similar, so I think we can keep using a unified mod compat template, and then label the issue once diagnosed accordingly.

Copy link
Author

Choose a reason for hiding this comment

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

Yep, you've separated mod malfunction and crash in the compat thread, and malfunction could be a user issue, besides, it is harder to investigate.

Choose a reason for hiding this comment

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

I agree with @Su5eD that those labels could be added by triage instead. I also noticed that the template now explicitly mentions modpacks - I think this is fine, but we should add a disclaimer of some sort that the user can use bisecting to identify which mod specifically causes problems. This process can take quite some time which makes plain "Modpack x is not working" issues not very actionable.

.github/ISSUE_TEMPLATE/mod_crash.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/mod_crash.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/suggestion.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/suggestion.yml Outdated Show resolved Hide resolved
@hexadecimal233
Copy link
Author

hexadecimal233 commented Dec 17, 2023

There has been no updates for months and we do see the increasingly high comment number in mod compat thread.
I also made this PR cuz mclo.gs provides a better log viewing experience and I've been wondering why it has not been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants