Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@Bibby0110
Copy link
Contributor

Reverts #12316

Less than 24 hours + a self merge (Admits it was his own idea)

@Hopekz
Copy link
Contributor

Hopekz commented Sep 17, 2021

Reverts #12316

Less than 24 hours + a self merge (Admits it was his own idea)

Where is the part where the PR is bad for the game since you're reverting it

@Bibby0110
Copy link
Contributor Author

Reverts #12316
Less than 24 hours + a self merge (Admits it was his own idea)

Where is the part where the PR is bad for the game since you're reverting it

Breaks protocol.
Doesn't have to be bad for the game.

@Hopekz
Copy link
Contributor

Hopekz commented Sep 17, 2021

Breaks protocol.
Doesn't have to be bad for the game.

Keep your dev drama out of our codebase.

Your change is modifying our codebase to remove something you don't even think is bad so by that logic you are knowingly making it worse if this were to be merged.

There are probably other ways you can express your point without making a modification like bringing it up in staff channels or even making a staff complaint on the forums.

@nesisknesis
Copy link
Contributor

why was this actually merged

@JamieD1 JamieD1 closed this Sep 19, 2021
@JamieD1 JamieD1 deleted the revert-12316-BS-microwaves branch September 20, 2021 10:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants