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

[MIRROR] moon smile effect durations fix #2250

Merged
merged 1 commit into from
Mar 6, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1270
Original PR: tgstation/tgstation#81794

About The Pull Request

Moon smile's durations had some syntax issues, causing their durations to be much lower than intended, and sanity having very low impact on it. Also adds a use of SANITY_MAXIMUM define, so that the calculation will still work if this value is ever changed.

Why It's Good For The Game

The moon did not smile. It was frowning. Frowning because it was bugged :( But with this fix it can smile again :)

Changelog

🆑 ViktorKoL
fix: fixed some issues when calculating the duration of moon smile's effects
/:cl:

* moon smile effect durations fix (#81794)

## About The Pull Request

Moon smile's durations had some syntax issues, causing their durations
to be much lower than intended, and sanity having very low impact on it.
Also adds a use of SANITY_MAXIMUM define, so that the calculation will
still work if this value is ever changed.
## Why It's Good For The Game

The moon did not smile. It was frowning. Frowning because it was bugged
:( But with this fix it can smile again :)
## Changelog
:cl:
fix: fixed some issues when calculating the duration of moon smile's
effects
/:cl:

* moon smile effect durations fix

---------

Co-authored-by: ViktorKoL <[email protected]>
@Iajret Iajret merged commit 6cc6220 into master Mar 6, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1270 branch March 6, 2024 17:31
AnywayFarus added a commit that referenced this pull request Mar 6, 2024
Iajret pushed a commit that referenced this pull request Apr 30, 2024
…utters on northstar (#2250)

* Fixes missing wires in tramstation, and fixes name of cmo shutters on northstar (#82955)

## About The Pull Request
This pr fixes wires not connecting to tramstatation graven and turbine
to smes
This pr fixes the name of cmo office shutter button from robotics to cmo

## Why It's Good For The Game
So power doesn't fail, and using the wrong names
## Changelog

:cl:
fix: Fixes tramstation gravgen wiring
fix: Fixes tramstation turbine wiring
fix: Fixes northstar cmo button name
/:cl:

* Fixes missing wires in tramstation, and fixes name of cmo shutters on northstar

---------

Co-authored-by: Nick <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants