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

fix the shadow factory #2299

Merged
merged 3 commits into from
Nov 28, 2024
Merged

Conversation

deltanedas
Copy link
Member

About the PR

borgs that were emagged before switching type lost emagged laws, now its fixed
ion storm still gets lost but oh well

Why / Balance

shadow factory in shambles

Technical details

special lawset handling instead of overwriting lawset blindly

Media

switched then emagged still works
07:48:27

emagged then switched, still has funny laws
07:48:49

Changelog
🆑

  • fix: Fixed borgs with special lawsets fixing their emag laws when switched to.

@deltanedas deltanedas requested review from a team as code owners November 27, 2024 07:49
@github-actions github-actions bot added Changes: YML Changes any yml files Changes: C# Changes any cs files Changes: Localization Changes any ftl files size/M 64-255 lines S: Needs Review labels Nov 27, 2024
@github-actions github-actions bot added the size/L 256-1023 lines label Nov 27, 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 S: Merge Conflict Fix your PR! and removed size/M 64-255 lines labels Nov 27, 2024
@github-actions github-actions bot added size/M 64-255 lines and removed S: Merge Conflict Fix your PR! size/L 256-1023 lines labels Nov 27, 2024
@deltanedas deltanedas merged commit b4d79a2 into DeltaV-Station:master Nov 28, 2024
13 of 14 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: Localization Changes any ftl files Changes: YML Changes any yml files S: Needs Review size/M 64-255 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants