Skip to content

add commands yml documentation#233

Merged
olijeffers0n merged 17 commits into
PaperMC:mainfrom
456dev:the456gamer-add-commands-yml
Dec 14, 2023
Merged

add commands yml documentation#233
olijeffers0n merged 17 commits into
PaperMC:mainfrom
456dev:the456gamer-add-commands-yml

Conversation

@456dev
Copy link
Copy Markdown
Contributor

@456dev 456dev commented Aug 29, 2023

also disable ligatures in config objects. only issue i can think of is if there is a code example in a config description, but that probably should have its own page anyway

Comment thread config/sidebar.paper.ts Outdated
Comment thread docs/paper/admin/reference/configuration/commands-configuration.mdx Outdated
Comment thread docs/paper/admin/reference/configuration/commands-configuration.mdx Outdated
Comment thread src/components/Config.tsx Outdated
Comment thread docs/paper/admin/reference/configuration/commands-configuration.mdx Outdated
Comment thread docs/paper/admin/reference/configuration/commands-configuration.mdx Outdated
Comment thread docs/paper/admin/reference/configuration/commands-configuration.mdx Outdated
Comment thread docs/paper/admin/reference/configuration/commands-configuration.mdx Outdated
@olijeffers0n
Copy link
Copy Markdown
Member

Still not a huge fan of "commands configuration" as a title, I feel like it makes it sound way more useful than it is... Not sure?

# Conflicts:
#	config/sidebar.paper.ts
rename slug to legacy-commands-configuration
rename file to bukkit-commands-configuration.mdx
rename title to Bukkit Commands Configuration
Comment thread docs/paper/admin/reference/configuration/bukkit-commands-configuration.mdx Outdated
Copy link
Copy Markdown
Member

@olijeffers0n olijeffers0n left a comment

Choose a reason for hiding this comment

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

LGTM Apart from this

Comment thread docs/paper/admin/reference/configuration/bukkit-commands-configuration.mdx Outdated
@olijeffers0n olijeffers0n merged commit f0cc172 into PaperMC:main Dec 14, 2023
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.

4 participants