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

Replaces exploit with gamebreaking exploit in PR request template #20707

Merged
merged 3 commits into from
Oct 24, 2023

Conversation

JohnFulpWillard
Copy link
Contributor

@JohnFulpWillard JohnFulpWillard commented Oct 22, 2023

Document the changes in your pull request

The PR template says what [s] should be used for, but just about any bug in the game can be classified as an exploit. I really don't want to have to stalk the github page to see these random PRs that fix issues that aren't breaking the game, and I think abusing it would also be bad.
I also mentioned that maintainers aren't aware of PRs with [s] so they can bring it up themselves to someone available. They do actually, I just haven't noticed it.

I've noticed a lot that [s] is abused frequently here, it's really stupid. Usually it's just to avoid commenters, which I do not agree with at all- stop trying to hide stuff from people god damn.

Not mentioning any PRs here because I don't want witch-hunts I just want this label to stop being used so liberally.

@JohnFulpWillard JohnFulpWillard requested a review from a team as a code owner October 22, 2023 09:41
@github-actions github-actions bot added the Github changes to the .github folder label Oct 22, 2023
@JohnFulpWillard JohnFulpWillard changed the title Replaces exploit with gamebreaking exploit in PR request template (+minor edit) Replaces exploit with gamebreaking exploit in PR request template Oct 22, 2023
@ToasterBiome
Copy link
Contributor

Lock conversation rubs hands together

@Addust
Copy link
Contributor

Addust commented Oct 23, 2023

is this because they secreted the AGCNR off-station should not gas station with fallout PR

@ToasterBiome ToasterBiome merged commit 86c2159 into yogstation13:master Oct 24, 2023
@cowbot92
Copy link
Contributor

it would've been funnier if you used [s] for this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Github changes to the .github folder
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants