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

feat: Automatic version update warnings + new config set cmd #354

Merged
merged 9 commits into from
May 27, 2022

Conversation

raulb
Copy link
Member

@raulb raulb commented May 13, 2022

Description of change

Type of change

  • New feature
  • Bug fix
  • Refactor
  • Documentation

How was this tested?

  • Unit Tests
  • Tested in staging
  • Tested in minikube

Demo

First time or once a week

# run any command
$ meroxa whoami 
[email protected]



  🎁 meroxa v2.1.1 is available! Update it by running: `brew upgrade meroxa`
  🧐 Check out latest changes in https://github.com/meroxa/cli/releases/tag/v2.1.1
  💡 To disable these warnings, run `meroxa config set DISABLE_NOTIFICATIONS_UPDATE=true`

Disable notifications on update

# we automatically convert keys to a screaming snake case (`disableNotificationsUpdate` -> `DISABLE_NOTIFICATIONS_UPDATE`)


$ meroxa config set disableNotificationsUpdate=true
Updating your Meroxa configuration file with DISABLE_NOTIFICATIONS_UPDATE=true...
Done!

Additional references

  • You can also update several keys at once:
$ meroxa config set disableNotificationsUpdate=true latestCliVersionUpdatedAt=
Updating your Meroxa configuration file with LATEST_CLI_VERSION_UPDATED_AT=...
Updating your Meroxa configuration file with DISABLE_NOTIFICATIONS_UPDATE=true...
Done!
  • This warning for updating CLI will never be shown when using --json to make sure we don't break any possible automation there.

Documentation updated

https://github.com/meroxa/meroxa-docs/pull/111

@raulb raulb self-assigned this May 13, 2022
@raulb raulb force-pushed the raul/autoupdate branch 2 times, most recently from 440bcdb to 1cd1edd Compare May 25, 2022 14:19
@raulb raulb force-pushed the raul/autoupdate branch 6 times, most recently from b02000d to a78e5c6 Compare May 26, 2022 20:42
@raulb raulb force-pushed the raul/autoupdate branch from a78e5c6 to 7445e82 Compare May 26, 2022 20:44
@raulb raulb force-pushed the raul/autoupdate branch from 7445e82 to 8224679 Compare May 26, 2022 20:58
raulb added 2 commits May 26, 2022 23:19
Now that we’re fetching from brew’s formula, we can compare directly with version defined by GoReleaser
@raulb raulb requested a review from jayjayjpg May 27, 2022 10:42
Copy link
Contributor

@jayjayjpg jayjayjpg left a comment

Choose a reason for hiding this comment

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

✨ This reads great, awesome work! ✨

@raulb raulb merged commit fcdf06d into master May 27, 2022
@raulb raulb deleted the raul/autoupdate branch May 27, 2022 12:34
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.

2 participants