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

fix checklist rendering in pr template #1418

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

benteg
Copy link
Contributor

@benteg benteg commented Aug 21, 2023

Description

Previously the checklist in the pr template used [x] ...

image

Type of change

❌ Bug fix (non-breaking change which fixes an issue)
✅ General change (non-breaking change that doesn't fit the above categories, such as copyediting)

@x9136
Copy link
Collaborator

x9136 commented Aug 22, 2023

Can someone tell me what effect you plan to achieve with this?

It's understandable that people will check the checkboxes without reading, but now you can rebuke them for not reading. Just why? I don't see the point of adding unnecessary actions to the dozens of actions you have to take to create and suggest an icon. I understand when everyone tries to make the process easier, not more complicated.

🙂

@SuperDragonXD SuperDragonXD merged commit 8948f0d into LawnchairLauncher:develop Aug 22, 2023
@SuperDragonXD
Copy link
Contributor

Please leave discussion regarding the checkboxes in #1413. This PR doesn't add the checkboxes, it only fixes them.

(Tbh, I don't feel like checking a box adds additional effort. It's a better reminder than a comment.)

@benteg benteg deleted the pr-template branch August 22, 2023 08:58
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