Skip to content

DOCS/man/commands: rename COMMANDS to CONSOLE COMMANDS#17958

Merged
Dudemanguy merged 1 commit into
mpv-player:masterfrom
guidocella:commands-script
Jun 17, 2026
Merged

DOCS/man/commands: rename COMMANDS to CONSOLE COMMANDS#17958
Dudemanguy merged 1 commit into
mpv-player:masterfrom
guidocella:commands-script

Conversation

@guidocella

Copy link
Copy Markdown
Contributor

Avoid confusing it with command interface.

@guidocella guidocella changed the title commands.rst: rename COMMANDS to COMMANDS SCRIPT DOCS/man/commands: rename COMMANDS to COMMANDS SCRIPT May 21, 2026
Comment thread DOCS/man/commands.rst Outdated
@@ -1,5 +1,5 @@
COMMANDS
========
COMMANDS SCRIPT

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Aren't those SCRIPT COMMANDS?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It doesn't define script commands though. It's a script to run commands.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry to bikeshed like a month later but how about CONSOLE COMMANDS?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Works for me.

Avoid confusing it with command interface.
@guidocella guidocella changed the title DOCS/man/commands: rename COMMANDS to COMMANDS SCRIPT DOCS/man/commands: rename COMMANDS to CONSOLE COMMANDS Jun 16, 2026
@Dudemanguy Dudemanguy merged commit 715d2b7 into mpv-player:master Jun 17, 2026
7 of 9 checks passed
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