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

Don't Require Running in Release to Test Mood #1257

Merged
merged 2 commits into from
Nov 30, 2024

Conversation

DEATHB4DEFEAT
Copy link
Member

No description provided.

@DEATHB4DEFEAT DEATHB4DEFEAT added Priority: 4-Low Should be resolved at some point Size: 5-Very Small For especially small issues/PRs Type: Bug [Un]intentional issue with something labels Nov 20, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added Status: Merge Conflict FIX YOUR PR AAAGH Changes: C# Changes any cs files Changes: YML Changes any yml files labels Nov 20, 2024
@DEATHB4DEFEAT
Copy link
Member Author

wtf conflicts?? I made sure to be up to date with master

@SimpleStation14 SimpleStation14 changed the title don't require running in release to test mood Don't Require Running in Release to Test Mood Nov 20, 2024
@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Nov 20, 2024
Comment on lines +2570 to +2574
#if RELEASE
CVarDef.Create("mood.enabled", true, CVar.SERVER);
#else
CVarDef.Create("mood.enabled", false, CVar.SERVER);
#endif
Copy link
Member

Choose a reason for hiding this comment

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

And now Mood doesn't work in Release configuration.

Copy link
Member Author

Choose a reason for hiding this comment

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

whar

Copy link
Member

Choose a reason for hiding this comment

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

In all instances of (!_config.GetCVar(CCVars.MoodEnabled)

Copy link
Member Author

Choose a reason for hiding this comment

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

why?

@FoxxoTrystan FoxxoTrystan requested review from a team, VMSolidus, Pspritechologist and FoxxoTrystan and removed request for a team November 21, 2024 15:47
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Nov 21, 2024
@VMSolidus VMSolidus merged commit f287188 into Simple-Station:master Nov 30, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: YML Changes any yml files Priority: 4-Low Should be resolved at some point Size: 5-Very Small For especially small issues/PRs Status: Needs Review Someone please review this Type: Bug [Un]intentional issue with something
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants