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

Add auto-update feature and improve code #7

Merged
merged 2 commits into from
Jan 20, 2019

Conversation

xilopaint
Copy link
Contributor

• Add auto update capability using OneUpdater
• Refactor workflowHandler.sh by using Alfred's environment variables
• Fix Shellcheck issues
• Minor changes in README

• Add auto update capability using OneUpdater
• Refactor workflowHandler.sh by using Alfred's environment variables
• Fix Shellcheck issues
• Minor changes in README
@xilopaint xilopaint changed the title Add auto update feature and improve code Add auto-update feature and improve code Jan 15, 2019
@xilopaint
Copy link
Contributor Author

In order to make the auto-update feature work you have to make a new release with the tag v1.1.

@mrodalgaard
Copy link
Owner

@xilopaint this looks really good. There is however still a failing unit test.

@xilopaint
Copy link
Contributor Author

xilopaint commented Jan 19, 2019

Sorry! I forgot to run the unit tests. Now everything is ok.

Also, I added support to DNS.WATCH and Cloudflare DNS servers.

@mrodalgaard
Copy link
Owner

Great job. I will push to a new release. Thank you.

@mrodalgaard mrodalgaard merged commit 360b224 into mrodalgaard:master Jan 20, 2019
@xilopaint xilopaint deleted the feat/general-improvements branch January 20, 2019 15:49
@xilopaint
Copy link
Contributor Author

Great job. I will push to a new release. Thank you.

There's an error in release notes. The new auto-update method doesn't depend on Packal. OneUpdater is a totally separate solution. Packal is no longer maintained and doesn't work for updates anymore.

@mrodalgaard
Copy link
Owner

Thank you for bringing this to my attention. I have updated the release notes.

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