Skip to content

Tooling Pairity with PvP and 516 Compat #797

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

Merged
merged 40 commits into from
Feb 25, 2025

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Feb 25, 2025

About the pull request

This PR ports most tooling differences between pve and pvp as well as 516 compatibility. This includes multivar support for hooks, additional map lints, additional grep tests, two additional action workflows, updates to mapmerge, and a means to easily ignore maps in CI.

Specific PRs ported via patch:

Explain why it's good for the game

PvE's tooling should not fall behind PvP and 516 is the future.

Testing Photographs and Procedure

See checks.

Changelog

🆑 Drathek HarryOB
code: Update most tooling including changes to hooks, mapmerge, additional grep checks, additional maplints, and additional workflows
code: 516 Compat
code: Added more OD pragmas and code fixes pertaining to them
maptweak: Fixed duplicate platforms on several maps
maptweak: Fixed plane edits on Golden Arrow and Almayer
/:cl:

@Drulikar Drulikar added the needs tested testmerge this to see what breaks label Feb 25, 2025
@github-actions github-actions bot added the UI label Feb 25, 2025
@Drulikar Drulikar changed the title Tooling Pairty with PvP Tooling Pairty with PvP and 516 Compat Feb 25, 2025
@Drulikar Drulikar marked this pull request as ready for review February 25, 2025 10:08
@Drulikar Drulikar added the Stale Exempt YOU MUST CONSTRUCT ADDITIONAL MAINTAINERS label Feb 25, 2025
@Drulikar Drulikar changed the title Tooling Pairty with PvP and 516 Compat Tooling Pairity with PvP and 516 Compat Feb 25, 2025
@AndroBetel AndroBetel merged commit b8916b7 into cmss13-devs:master Feb 25, 2025
40 checks passed
cm13-github added a commit that referenced this pull request Feb 25, 2025
@Drulikar Drulikar mentioned this pull request Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Github Mapping needs tested testmerge this to see what breaks Stale Exempt YOU MUST CONSTRUCT ADDITIONAL MAINTAINERS UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants