Skip to content

Minor Drozd Code Oopsie#4308

Open
MagosAmphrose wants to merge 13 commits intonew-frontiers-14:masterfrom
MagosAmphrose:Drozd-Is-Cool
Open

Minor Drozd Code Oopsie#4308
MagosAmphrose wants to merge 13 commits intonew-frontiers-14:masterfrom
MagosAmphrose:Drozd-Is-Cool

Conversation

@MagosAmphrose
Copy link
Contributor

About the PR

Minor error in my previous code, pointed out to me courtesy of Duran working through their merge conflicts.

Why / Balance

Because even if it works without it fixing it anyways is probably a good idea.

Technical details

Added two lines for uplink-security-drozd-name and uplink-security-drozd-desc.

How to test

Obtain uplink as Bailiff/Sheriff.
Observe that the Drozd has an updated entry.

Media

Requirements

Changelog

Copy link
Contributor

@mnva0 mnva0 left a comment

Choose a reason for hiding this comment

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

Thank you for this PR! Not a maintainer, this is just something I noticed

uplink-security-vulcan-name = Vulcan
uplink-security-vulcan-desc = Heavy duty, select fire battle rifle. Uses .30 rifle.

uplink-security-drozd-name= Drozd
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably have a space on both sides of the =

@ErhardSteinhauer ErhardSteinhauer added S: Awaiting Changes This PR has changes that need to be made before merging and removed S: Untriaged labels Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FTL No C# S: Awaiting Changes This PR has changes that need to be made before merging S: Needs Review This PR is awaiting reviews size/XS YML

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants