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

Set up CI with Azure Pipelines #2

Open
wants to merge 156 commits into
base: master
Choose a base branch
from
Open

Conversation

azure-pipelines[bot]
Copy link

No description provided.

jlipps and others added 30 commits February 8, 2018 16:32
* Add check-engine to package-ci script
* Require NPM 3 -> 4 and Node 6 -> 7
  * This is to prevent problems we had when using newer versions of NPM/Node
  * Since this isn't a package that is published to NPM and is not ever imported as a dependency, having strict constraints isn't a big problem
* Fix e2e because it uses nested node_modules when using shrinkwrap
* Use request to check if updates available

* Native autoUpdater downloads updates automatically
* We want to be able to check if updates are available first
* So, make a call to /update endpoint directly and pass in the version to check if there's an update available

* Redo autoupdater

* Use the native electron dialogs instead of using a custom React workflow
* Right now it automatically downloads it
* Next step will be to check for availability first before proceeding to download

* Show message if no updates available
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.

9 participants