Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

There should only be one list of command-line options #618

Open
Jayman2000 opened this issue Oct 5, 2024 · 2 comments
Open

There should only be one list of command-line options #618

Jayman2000 opened this issue Oct 5, 2024 · 2 comments
Milestone

Comments

@Jayman2000
Copy link
Contributor

At the moment, both USAGE.md and the wiki have lists of command-line options for Descent 3. It would be better if there was just one list instead of two.

In the PR description for #471, I said that I would open a pull request that adds documentation for -additionaldir if #471 got merged. Now that #471 has been merged, it’s not clear to me if I should add the documentation to USAGE.md , the wiki or both. If there was only one list of command-line options, then it would be clear where documentation for new command-line options should go.

@winterheart
Copy link
Collaborator

USAGE.md is main doc for users, cmd-line options included. New documentation related to user interaction should be reflected in USAGE.md.

@Lgt2x
Copy link
Member

Lgt2x commented Oct 6, 2024

The wiki lists the command-line options available when the project started. Since then, several have been removed and added. The source of truth should be USAGE.md, and the wiki should be used for developer docs.

It could be its own issue, but the game itself should know what options it supports and be able to display a help menu when launched with --help/-h.

@Lgt2x Lgt2x added this to the 1.6 milestone Oct 6, 2024
Jayman2000 added a commit to Jayman2000/Descent3-pr that referenced this issue Oct 6, 2024
This commit takes all of the command-line options documentation from the
wiki [1] and merges it with the command-line options documentation
that’s in USAGE.md. This change will eventually allow us to remove the
command-line options documentation from the wiki so that we only have
one list of command-line options instead of two.

Before this change, USAGE.md used a Markdown table and the wiki used an
HTML table. I ended up replacing the Markdown table in USAGE.md with an
HTML table because it was easier to copy and paste the HTML table from
the wiki than to convert the HTML table from the wiki into a Markdown
table.

The main motivation behind this change is to make it easier to create a
future commit. That future commit will add documentation for the
-additionaldir command-line option. Making sure that there’s only one
list of command-line options instead of two will make it so that
-additionaldir doesn’t need to be documented in two different places.

This is related DescentDevelopers#618, but it doesn’t fully fix the issue.

[1]: <https://github.com/DescentDevelopers/Descent3/wiki/Running-Descent-3#command-line-options>
Jayman2000 added a commit to Jayman2000/Descent3-pr that referenced this issue Oct 6, 2024
This commit takes all of the command-line options documentation from the
wiki [1] and merges it with the command-line options documentation
that’s in USAGE.md. This change will eventually allow us to remove the
command-line options documentation from the wiki so that we only have
one list of command-line options instead of two.

Before this change, USAGE.md used a Markdown table and the wiki used an
HTML table. I ended up replacing the Markdown table in USAGE.md with an
HTML table because it was easier to copy and paste the HTML table from
the wiki than to convert the HTML table from the wiki into a Markdown
table.

The main motivation behind this change is to make it easier to create a
future commit. That future commit will add documentation for the
-additionaldir command-line option. Making sure that there’s only one
list of command-line options instead of two will make it so that
-additionaldir doesn’t need to be documented in two different places.

This is related DescentDevelopers#618, but it doesn’t fully fix that issue.

[1]: <https://github.com/DescentDevelopers/Descent3/wiki/Running-Descent-3#command-line-options>
Jayman2000 added a commit to Jayman2000/Descent3-pr that referenced this issue Oct 20, 2024
This commit takes all of the command-line options documentation from the
wiki [1] and merges it with the command-line options documentation
that’s in USAGE.md. This change will eventually allow us to remove the
command-line options documentation from the wiki so that we only have
one list of command-line options instead of two.

Before this change, USAGE.md used a Markdown table and the wiki used an
HTML table. I ended up replacing the Markdown table in USAGE.md with an
HTML table because it was easier to copy and paste the HTML table from
the wiki than to convert the HTML table from the wiki into a Markdown
table.

The main motivation behind this change is to make it easier to create a
future commit. That future commit will add documentation for the
-additionaldir command-line option. Making sure that there’s only one
list of command-line options instead of two will make it so that
-additionaldir doesn’t need to be documented in two different places.

This is related DescentDevelopers#618, but it doesn’t fully fix that issue.

[1]: <https://github.com/DescentDevelopers/Descent3/wiki/Running-Descent-3#command-line-options>
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

No branches or pull requests

3 participants