Skip to content

New NoAmmoWeapons tag supporting multiple weapon types - #2429

Merged
Coronia merged 4 commits into
Phobos-developers:developfrom
Fly-Star-him:noammoweapons
Oct 6, 2026
Merged

Coronia merged 4 commits into
Phobos-developers:developfrom
Fly-Star-him:noammoweapons

Conversation

@Fly-Star-him

@Fly-Star-him Fly-Star-him commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

What kind of change is this?

  • New feature, vanilla bugfix or enhancement of a released feature - changelog, docs and credits entries are needed.
  • Improvement to a new (unreleased) feature - docs and credits entries are needed; no changelog entry (Skip Changelog).
  • Bugfix to a new (unreleased) feature - credits entry is needed; no changelog or docs entries (Skip Changelog, Skip Docs).
  • Bugfix to an old (released) feature - changelog and credits entries are needed; no docs entry (Skip Docs).
  • Completely minor change (e.g. a typo fix) - no entries are needed (Skip Changelog, Skip Docs, Skip Credits).

Since the author is me, please apply Skip Credits label.

Description

  • You can now use multiple weapons with NoAmmoWeapons while NoAmmoAmount is active.
  • NoAmmoWeapons selects an appropriate weapon based on the weapon's CanTarget and other tags, as well as the warhead's Verses.
  • If no suitable weapon is found, it continues to use NoAmmoWeapon.

In rulesmd.ini:

[SOMETECHNO]                        ; TechnoType
NoAmmoWeapons=                      ; List of integers
NoAmmoWeapons.IgnoreNeverUse=yes    ; boolean

  • 现在你可以在NoAmmoAmount生效时使用NoAmmoWeapons指定多个武器。

    • NoAmmoWeapons会根据武器的CanTarget等各项标签以及弹头的Verses来检查当前武器是否可用。
    • 未能搜寻到合适的武器则继续使用NoAmmoWeapon。

@phoboscn-bot

Copy link
Copy Markdown

To Chinese users:
This pull request has been mentioned on Phobos CN. There might be relevant details there:

致中文用户:
此拉取请求已在 Phobos CN 上被提及。那里可能有相关详细信息:

https://www.phoboscn.top/t/topic/788/1

@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

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.

@DeathFishAtEase DeathFishAtEase added Needs testing ⚙️T1 T1 maintainer review is sufficient labels Oct 2, 2026
@Coronia Coronia added Tested Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions. and removed Needs testing labels Oct 5, 2026
@Coronia
Coronia merged commit 183eca0 into Phobos-developers:develop Oct 6, 2026
7 checks passed
DeathFishAtEase added a commit that referenced this pull request Oct 6, 2026
…ced by #2429

Preliminarily judged that the issue was introduced by commit 1074043 when syncing develop, and it is speculated that the documentation changes of the develop branch were not correctly synced when resolving conflicts.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️T1 T1 maintainer review is sufficient Tested Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants