Skip to content

List KSArmory - #37

Open
LaurensDeV wants to merge 1 commit into
KSAModding:mainfrom
LaurensDeV:list-ksarmory
Open

List KSArmory#37
LaurensDeV wants to merge 1 commit into
KSAModding:mainfrom
LaurensDeV:list-ksarmory

Conversation

@LaurensDeV

Copy link
Copy Markdown

Adds listings/KSArmory.toml — weapon systems for KSA: point defence, guided and unguided munitions, targeting sights and a ballistic computer.

forums https://forums.ahwoo.com/forums/kitten-space-agency/mod-releases/kessler-systems-armory-ksarmory.1131/
repository https://github.com/LaurensDeV/KSArmory (MIT)
SpaceDock https://spacedock.info/mod/4460/KSArmory
latest release 0.8.44

Ownership should verify by rule 1 — the repository in [releases] is my own account and not a fork. SpaceDock is listed alongside it, with authority = "github".

Checked locally before opening:

check_layout   exit 0
check_schema   exit 0
check_index    exit 0
check_license  exit 0
check_status   exit 0
check_release  listings/KSArmory.toml: pass: the latest release 0.8.44 stamps cleanly

One thing to flag: the tags are ["parts", "weapons", "physics"], and weapons may not be in the curated vocabulary. Happy to drop it if it warns.

@MrJeranimo

MrJeranimo commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

One thing to flag: the tags are ["parts", "weapons", "physics"], and weapons may not be in the curated vocabulary. Happy to drop it if it warns.

That should be fine. The currated vocabulary, as far as I know, is mainly for a "Select Tags" dropdown/list with an optional "Search by Tag", so the weapons may just not show in the dropdown/list and the user has to manually search the tag name.

@MrJeranimo MrJeranimo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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.

2 participants