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

tidy electron files, they weren't pwetty #3993

Merged
merged 5 commits into from May 26, 2017
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.

Always

Just for now

missed a concat :)

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
  • Loading branch information
t3chguy committed May 22, 2017
commit 8994c2bad14294a54986bf2f4d220b0d1e85b050
@@ -128,7 +128,7 @@ function startAutoUpdate(updateBaseUrl) {
`${updateBaseUrl}macos/?localVersion=${encodeURIComponent(electron.app.getVersion())}`);

This comment has been minimized.

Copy link
@ara4n

ara4n May 26, 2017

Member

i'm not sure this is a legibility improvement tbh...


} else if (process.platform === 'win32') {
electron.autoUpdater.setFeedURL(updateBaseUrl + 'win32/' + process.arch + '/');
electron.autoUpdater.setFeedURL(`${updateBaseUrl}win32/${process.arch}/`);
} else {
// Squirrel / electron only supports auto-update on these two platforms.
// I'm not even going to try to guess which feed style they'd use if they
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.