Skip to content

use verb macro for defining some admin verbs + realias tg ones#7508

Open
LetterN wants to merge 3 commits intoCitadel-Station-13:masterfrom
LetterN:verbify-all
Open

use verb macro for defining some admin verbs + realias tg ones#7508
LetterN wants to merge 3 commits intoCitadel-Station-13:masterfrom
LetterN:verbify-all

Conversation

@LetterN
Copy link
Contributor

@LetterN LetterN commented Jan 26, 2026

About The Pull Request

Adds tg ADMIN_VERB alias among other things (i just want to use their naming scheme). verb__invoke_##PATH_SUFFIX(client/user, ##HEADER) is now user instead of invoking

also debug_controller now does not lie, it debugs controllers not subsystems.

Why It's Good For The Game

Changelog

@github-actions github-actions bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. Config Update and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 26, 2026
@LetterN LetterN changed the title use verb macro for defining verbs + add tg one use verb macro for defining admin verbs + realias tg ones Jan 26, 2026
@LetterN LetterN changed the title use verb macro for defining admin verbs + realias tg ones use verb macro for defining some admin verbs + realias tg ones Jan 26, 2026
@@ -0,0 +1,40 @@
// Re-aliased tg defines
Copy link
Contributor

Choose a reason for hiding this comment

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

don't realias macros if it just does the same thing
for context only just add a def that just nulls it

also, there's no need to replace the verb categories unless we're going to get a statpanel that doesn't treat them as verbs; they're all verbs internally, thus they are verb categories.

stuff like profile/ipintel should not be their own tabs, statpanel if needed should just parse the category like Category.Subcategory or something

rest of PR is fine but this is duplicate

Copy link
Contributor Author

Choose a reason for hiding this comment

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

license
also category is just a rename

@github-actions
Copy link

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ two week, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale label Feb 11, 2026
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 11, 2026
@silicons silicons added Stale Exempt Immune to Stale and removed Stale labels Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Config Update size/L Denotes a PR that changes 100-499 lines, ignoring generated files. Stale Exempt Immune to Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants