Skip to content

[Minor] Add global default values for some tags#2165

Open
Coronia wants to merge 1 commit intoPhobos-developers:developfrom
Coronia:global-tag
Open

[Minor] Add global default values for some tags#2165
Coronia wants to merge 1 commit intoPhobos-developers:developfrom
Coronia:global-tag

Conversation

@Coronia
Copy link
Copy Markdown
Contributor

@Coronia Coronia commented Apr 7, 2026

If there's other tag that is worth globalizing please let me know

@Coronia Coronia added Minor Minor feature and/or fix, not a lot of changes or they are not significant ⚙️T1 T1 maintainer review is sufficient No Documentation Needed No documentation needed whatsoever No test needed This PR is simple enough, or changes no in-game logic, so no in-game testing is required. labels Apr 7, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@TaranDahl
Copy link
Copy Markdown
Contributor

Arcing.AllowElevationInaccuracy.
CanBeBuiltOn and IsPassable for TerrainTypes. This needs a smart default, smart to true for non-Tiber tree.
Splits.UseWeaponTargeting, AirburstWeapon.ApplyFirepowerMult
Shrapnel.AffectsGround, Shrapnel.AffectsBuildings, Shrapnel.UseWeaponTargeting

Copy link
Copy Markdown
Contributor

@TaranDahl TaranDahl left a comment

Choose a reason for hiding this comment

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

It would be better to wait for a longer period of time and take into account more users' opinions.

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

Labels

Minor Minor feature and/or fix, not a lot of changes or they are not significant No Documentation Needed No documentation needed whatsoever No test needed This PR is simple enough, or changes no in-game logic, so no in-game testing is required. ⚙️T1 T1 maintainer review is sufficient

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants