Skip to content

Update USAGE.md to enhance command documentation#4855

Open
tboy1337 wants to merge 1 commit into
cabaletta:1.19.4from
tboy1337:edit
Open

Update USAGE.md to enhance command documentation#4855
tboy1337 wants to merge 1 commit into
cabaletta:1.19.4from
tboy1337:edit

Conversation

@tboy1337
Copy link
Copy Markdown

No description provided.

Comment thread USAGE.md Outdated
Comment thread USAGE.md
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
@tboy1337 tboy1337 requested a review from Murat65536 August 17, 2025 11:16
@tboy1337
Copy link
Copy Markdown
Author

tboy1337 commented Sep 5, 2025

@leijurv LGTM?

@tboy1337
Copy link
Copy Markdown
Author

?

Copy link
Copy Markdown
Collaborator

@ZacSharp ZacSharp left a comment

Choose a reason for hiding this comment

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

There are clearly some good things in this pr and updating the documentation is a good idea, but I do not think this is ready for merge.

The list of things in this document is too long and a lot of the information is just duplicated from elsewhere and will inevitably go out of sync. To help newcomers we really do not need more than a very rough overview, the ingame help and settings documentation already have complete lists.

Also while reading this the second time, I started to wonder who's spent more time on this. There are some things in here which I do not want to think you wrote, but assuming you didn't write the text doesn't feel any better.

Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
@tboy1337 tboy1337 requested a review from Murat65536 November 29, 2025 02:10
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
@tboy1337 tboy1337 requested a review from Murat65536 November 29, 2025 03:10
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md
Comment thread USAGE.md
Comment thread USAGE.md Outdated
Comment thread USAGE.md Outdated
Comment thread USAGE.md
@tboy1337 tboy1337 requested a review from Murat65536 December 1, 2025 10:11
@tboy1337 tboy1337 force-pushed the edit branch 2 times, most recently from 883ced4 to d5ce4ec Compare January 18, 2026 06:42
@tboy1337
Copy link
Copy Markdown
Author

@leijurv I have rebased

@tboy1337
Copy link
Copy Markdown
Author

@ZacSharp @Murat65536 @leijurv there are no longer any merge conflicts.

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.

3 participants