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

Remove all command age requirements #32340

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

VasilisThePikachu
Copy link
Member

@VasilisThePikachu VasilisThePikachu commented Sep 21, 2024

About the PR

Removes the age requirements from command jobs

Why / Balance

From what I can tell this was merged with this on accident.

Maintainers see the maint review thread on this.

The original pr also claimed this was as an example. #30347

In my opinion wizden should not have an age requirement for command.

Note to everyone: FORKS CAN REVERT THIS, this pr is being made for wizard den and targeting the LRP servers

Requirements

Changelog

🆑 Vasilis

  • tweak: Removed the age requirement for command jobs.

From what I can tell this was merged with this on accident.

Maintainers see the maint review thread on this.

The original pr also claimed this was as an example.

In my opinion wizden should not have an age requirement for command.
@github-actions github-actions bot added the No C# For things that don't need code. label Sep 21, 2024
@Ertanic
Copy link
Contributor

Ertanic commented Sep 21, 2024

Maybe you should have just made it a matter of disabling the age restriction via cvar? If some fork really doesn't need that.

@VasilisThePikachu
Copy link
Member Author

VasilisThePikachu commented Sep 21, 2024

Maybe you should have just made it a matter of disabling the age restriction via cvar?

Are you concerned about forks? Forks may revert this with git if it gets merged.

Also, a cvar does not really make sense here.

@VasilisThePikachu VasilisThePikachu added the Undergoing Maintainer Discussion This PR is currently going through the 72-hour discussion window as per maintainer policy label Sep 21, 2024
@superjj18

This comment was marked as off-topic.

@Moomoobeef

This comment was marked as off-topic.

@0x6273
Copy link
Contributor

0x6273 commented Sep 21, 2024

From what I can tell this was merged with this on accident.

No, that PR has a changelog entry so it's definitely not an accident.

In my opinion wizden should not have an age requirement for command.

"In my opinion" is not a very good explanation for making a change like this. Could you explain why do you hold that opinion?

@VasilisThePikachu
Copy link
Member Author

VasilisThePikachu commented Sep 22, 2024

From what I can tell this was merged with this on accident.

No, that PR has a changelog entry so it's definitely not an accident.

And it was in an RP way, which goes against our policy. Anyhow, in the pr description you can see how it's explained to be "As an example". Internal maintainer review noticed this as well after being merged. With the idea that it would be fine to revert this if it ended up being a problem. (This chatter was between 2 people. The only people reviewing it)

Along with that, something else was amiss during the review of this pr, which I will cover next

In my opinion wizden should not have an age requirement for command.

"In my opinion" is not a very good explanation for making a change like this. Could you explain why do you hold that opinion?

Now yes I made this pr with a headache and it was late. So let me explain further.

In upstream as of currently we do not want to restrict player creation. And as the maintainer review is going currently. All participating maintainers agree.

Yes, it is silly that an 18-year-old can be a captain from a rp standpoint, which from a quick chat (And also recognizing some names) I had seems to be why most of the downvotes are there. Most of our servers are LRP and we do not have a lore. And it feels like a weird limitation to have on Wizard den servers. Hell, even then. I wouldn't mind seeing someone explain how to get captain at the age of 18. It gets people role-playing and creating a story. I could be stretching this here, but i would believe this falls under the seriously silly pillar

The second part of this story is that, this feature getting into upstream was not even talked with other maintainers. The species team (which I'm part of) was NOT notified about this change. I only found out after someone asked about it in the general chat on discord.

Normally, we use the work groups system to alert maintainers that are interested in a certain part of the game about changes and to give their opinions. This did not happen, I do not blame anyone for the slip up. We are human, after all. But I believe now that we have noticed this, we believe that it should have not made it into the game.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No C# For things that don't need code. Undergoing Maintainer Discussion This PR is currently going through the 72-hour discussion window as per maintainer policy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants