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

[CLI] Update docs and add cli versioning #218

Merged
merged 7 commits into from
Nov 8, 2023
Merged

[CLI] Update docs and add cli versioning #218

merged 7 commits into from
Nov 8, 2023

Conversation

jhnstn
Copy link
Member

@jhnstn jhnstn commented Nov 8, 2023

This updates the docs to reflect the new CLI tool.
It also introduces a version check to the cli tool

Highlights

  • Consolidate the Releasing.md files
  • Add more information on installing and using the cli tool
  • Add a warning if using an outdated version of the cli
  • Remove the bin directory in favor of adding binaries to the releases

Testing

  • Run GBM_WPMOBILE_ORG=wordpress-mobile go run main.go --version
  • Expect to see a warning about the version: "[WARN] You are running an older version of the CLI. Please update to v1.1.0"
  • Verify that the version printed is "v1.2.0"

Note: The message might seem off since the published version is behind the one in this PR. This is expected since this PR first introduces the cmd versioning. Once this is merged, a new release will be created and the message will make more sense.

Copy link
Contributor

@derekblank derekblank left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for updating the docs. 🚀

@jhnstn jhnstn merged commit d000c32 into trunk Nov 8, 2023
2 checks passed
@jhnstn jhnstn deleted the cli/update-docs branch November 8, 2023 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants