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

Name Changes Updates #5481

Merged
merged 2 commits into from
May 19, 2024
Merged

Name Changes Updates #5481

merged 2 commits into from
May 19, 2024

Conversation

SJuliez
Copy link
Member

@SJuliez SJuliez commented May 17, 2024

Adds a further test to the NameChangesValidator (checks to see if an entry has equal left and right side making it useless) and adds some comments. Also cleans up name-changes.txt, removing all problems found with the validator.

@SJuliez SJuliez added the Data Hammertime. label May 17, 2024
Copy link
Collaborator

@Sleet01 Sleet01 left a comment

Choose a reason for hiding this comment

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

LGTM

@HammerGS HammerGS merged commit 06698c6 into MegaMek:master May 19, 2024
4 checks passed
HammerGS added a commit that referenced this pull request May 19, 2024
+ PR #5487: Accessibility window cleanup and safety
+ Fix 5482: Process weapon quirks correctly in loading/saving protomeks #5484
+ PR #5481: Name Changes Updates
+ PR #5466: Move Field of Fire Sprites out of BoardView
+ PR #5472: Use FlatLaf exclusively
@SJuliez SJuliez deleted the name-changes branch May 19, 2024 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Data Hammertime.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants