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

fix: update mo2 'getprefix' arg to match help docs #1099

Merged
merged 1 commit into from Apr 28, 2024

Conversation

koppchen
Copy link
Contributor

Small fix to change the mo2 getpfx option to mo2 getprefix, which matches what's in the howto/help docs.

I chose this route over updating the docs since it matched the vortex getprefix|gp args, but feel free to change it.

Testing (before)

Show that the howto/help docs list mo2 getprefix|gp options. (this PR doesn't change this)

$ steamtinkerlaunch --help | grep -A 3 " mo2"
    mo2                              Mod Organizer 2
         create-instance|ci <id>     create MO2 instance for <id>
         download|d                  download latest MO2 release
         getprefix|gp                Output path to MO2 prefix

Using steamtinkerlaunch mo2 getprefix displays the howto/help docs

$ steamtinkerlaunch mo2 gp
/home/user/.config/steamtinkerlaunch/mo2/compatdata/pfx

$ steamtinkerlaunch mo2 getprefix | head -n 4
=========================
SteamTinkerLaunch v14.0.20240423-1
=========================
Usage: steamtinkerlaunch [options]...

Testing (after)

Using steamtinkerlaunch mo2 getprefix matches the mo2 gp output.

$ steamtinkerlaunch mo2 gp
/home/user/.config/steamtinkerlaunch/mo2/compatdata/pfx

$ steamtinkerlaunch mo2 getprefix
/home/user/.config/steamtinkerlaunch/mo2/compatdata/pfx

@sonic2kk
Copy link
Owner

This looks like a silly mistake on my part. Thanks for fixing it! Will merge

@sonic2kk sonic2kk merged commit 8de18cc into sonic2kk:master Apr 28, 2024
1 check passed
@sonic2kk
Copy link
Owner

oops, meant to bump the version. Will push another commit to do it.

This fix won't be noted on the changelog because it fixes something not yet in a release, I hope that's okay. Thank you very much for the contribution nonetheless!

sonic2kk added a commit that referenced this pull request Apr 28, 2024
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.

None yet

2 participants