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

Clarify wording on borgs NLR rules #2161

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

ewokswagger
Copy link
Contributor

About the PR

Clarified that borgs may choose whether they forget things upon being removed from a chassis or not.

Why / Balance

Rules as written imply borgs are prohibited to roleplay forgetting things due to being removed from a chassis. The new wording removes that implication. The Intention of the ruling was clarified in the discord.

Technical details

Minor change

Media

N/A

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Breaking changes

no

Changelog
No cl

@github-actions github-actions bot added the Changes: Documentation Changes any xml or md files label Nov 9, 2024
@ewokswagger
Copy link
Contributor Author

@Colin-Tel You were the one to provide the clarification so looping you in to check the new wording.

Co-authored-by: Colin-Tel <[email protected]>
Signed-off-by: SolStar <[email protected]>
@github-actions github-actions bot added the size/XS Under 16 lines label Nov 24, 2024
Copy link
Contributor

@Colin-Tel Colin-Tel left a comment

Choose a reason for hiding this comment

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

ShiftySam approves
image

@Colin-Tel Colin-Tel enabled auto-merge (squash) November 24, 2024 23:49
@ewokswagger
Copy link
Contributor Author

Now we wait for a heizenbug :)

@deltanedas
Copy link
Member

need headmin approval still @therealDLondon

@MilonPL
Copy link
Member

MilonPL commented Nov 25, 2024

need headmin approval still @therealDLondon

colin is a headmin too

@deltanedas
Copy link
Member

oh forgor

@deltanedas deltanedas disabled auto-merge November 25, 2024 05:43
@deltanedas deltanedas merged commit 02f4007 into DeltaV-Station:master Nov 25, 2024
11 of 12 checks passed
@ewokswagger ewokswagger deleted the nlr-clarification branch November 25, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Documentation Changes any xml or md files S: Needs Review size/XS Under 16 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants