Skip to content

Add Electron upgrade checks to Electronegativity#60

Merged
phosphore merged 2 commits into
doyensec:masterfrom
jkleinsc:electron-upgrades
Mar 25, 2020
Merged

Add Electron upgrade checks to Electronegativity#60
phosphore merged 2 commits into
doyensec:masterfrom
jkleinsc:electron-upgrades

Conversation

@jkleinsc

@jkleinsc jkleinsc commented Mar 4, 2020

Copy link
Copy Markdown
Contributor

This PR adds the option to run Electron upgrade checks with Electronegativity. The intention of this feature is for developers to discover the breaking changes that will affect them when they upgrade to a newer version of Electron.

As it currently stands this PR adds checks for Electron versions 5-8.

To use this feature a new argument, -u or --upgrade is used. When using this argument, the user will need to pass along a string in the format of x..y where x is the user's current version of Electron and y is the user's target version of Electron to upgrade to. For example for an app that is currently using Electron 4 and trying to upgrade to Electron 8, electronegativity would be invoked as follows:
electronegativity -i /path/to/electron/app -v -u 4..8

@ikkisoft ikkisoft requested a review from phosphore March 4, 2020 20:58
@phosphore

Copy link
Copy Markdown
Contributor

Hey @jkleinsc,
Great contributions! Thank you for your effort :)
Would you mind adding a few tests for the -u flag? Then it's good to go for me!

@jkleinsc

jkleinsc commented Mar 9, 2020

Copy link
Copy Markdown
Contributor Author

@phosphore sounds good.

@jkleinsc

Copy link
Copy Markdown
Contributor Author

@phosphore it took me a while to get back to this, but I've now added tests.

@phosphore

phosphore commented Mar 25, 2020

Copy link
Copy Markdown
Contributor

Love this! C:

@phosphore phosphore merged commit b7a40b3 into doyensec:master Mar 25, 2020
@jkleinsc jkleinsc deleted the electron-upgrades branch March 25, 2020 17:10
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