Skip to content

Fix typo in DoorName enum#374

Open
loicsmith wants to merge 1 commit intonorthwood-studios:masterfrom
loicsmith:patch-1
Open

Fix typo in DoorName enum#374
loicsmith wants to merge 1 commit intonorthwood-studios:masterfrom
loicsmith:patch-1

Conversation

@loicsmith
Copy link

small change to the DoorName enum correcting a typo.

Hcz106Primiary --> Hcz106Primary

Copilot AI review requested due to automatic review settings March 22, 2026 10:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to correct a typo in the public DoorName enum (Hcz106PrimiaryHcz106Primary) to match the intended door tag naming.

Changes:

  • Renames the DoorName enum member for the HCZ 106 primary door from Hcz106Primiary to Hcz106Primary.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -33,7 +33,7 @@ public enum DoorName
HczHidUpper, // HID_UPPER
HczHidLower, // HID_LOWER
HczNukeArmory, // NUKE_ARMORY
Copy link

Copilot AI Mar 22, 2026

Choose a reason for hiding this comment

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

Renaming this enum member will break existing references. The codebase still refers to DoorName.Hcz106Primiary (e.g., LabApi/Features/Wrappers/Facility/Doors/Door.cs maps "106_PRIMARY" to that value), so this change will currently fail to compile unless those references are updated. If this enum is part of the public API, consider keeping the old misspelled member as an [Obsolete] alias assigned to the same underlying value to avoid a breaking change for downstream consumers.

Suggested change
HczNukeArmory, // NUKE_ARMORY
HczNukeArmory, // NUKE_ARMORY
[System.Obsolete("Use Hcz106Primary instead.")]
Hcz106Primiary = Hcz106Primary,

Copilot uses AI. Check for mistakes.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is there dumbass copilot

Copy link
Author

Choose a reason for hiding this comment

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

i don't know..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants