-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add auto update command #557
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Itxaka <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #557 +/- ##
==========================================
+ Coverage 48.10% 48.20% +0.09%
==========================================
Files 48 48
Lines 6153 6153
==========================================
+ Hits 2960 2966 +6
+ Misses 2912 2908 -4
+ Partials 281 279 -2 ☔ View full report in Codecov by Sentry. |
I guess the configuration would need a URL for the repository, no? And some versioning checks to know what's new and what's not? |
I thin it picks everything from the os-release file. the version and the original repo it was installed from so it would mostly work out of the box on existing installs. Not tested :P Edit: yes it does. Same thing as the list-releases subcommand, it uses the existing data to get the list of releases from the remote repo. In this case, it just calls the upgrade afterwards :) |
We dont fail out as if there is not newer releases we dont really care? Signed-off-by: Itxaka <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 😁
Requested by @mauromorales :P