-
-
Notifications
You must be signed in to change notification settings - Fork 355
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: check update by release version, minor fixes #155
Conversation
jackblk
commented
Dec 3, 2021
- use github release api to check for version
- remove unused check-update-github package
- fix uncaught exception when checking for version SyntaxError: Unexpected end of JSON input #152
- update gitignore & dockerignore
- refactor dockerfile & bump apprise version
* use github release api to check for version * remove unused check-update-github package * fix uncaught exception when checking for version Revadike#152 * update gitignore & dockerignore * refactor dockerfile & bump apprise version
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.
Thanks for the PR. I left some suggestions.
@Revadike updated the PR. Thanks for the suggestions, I think it looks better now. |
I was just thinking, maybe we add an auto-update module instead? That way we can push fixes and people won't even have to manually update it. @jackblk |
@Revadike Good idea, for node user they will need to update manually so this will help. However, if they use Docker container then it is already auto update if I push a new version. |