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

Updated dependencies to the latest version #2146

Merged
merged 10 commits into from
Aug 31, 2017
Merged

Updated dependencies to the latest version #2146

merged 10 commits into from
Aug 31, 2017

Conversation

leo
Copy link
Contributor

@leo leo commented Aug 31, 2017

There comes a time for any big project where you need to ensure that old versions of your dependencies don't get dragged along until infinity. IMO, we just reached that point for Hyper (the longer we wait, the more is piling up).

I'm carefully testing this locally to ensure that everything is working afterwards and I suggest you (as a reader) do that as well, if you have time.

@leo leo requested review from chabou and albinekb August 31, 2017 12:17
@@ -55,8 +55,6 @@ const isDev = require('electron-is-dev');
const AppMenu = require('./menus/menu');
const config = require('./config');

app.commandLine.appendSwitch('js-flags', '--harmony-async-await');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With this removed, I assume electron runs node 8 now? then we should change travis to reflect the same node version (so that tests run the same node version) (...for when we have tests 😄 )

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup. Done! e0bb8b7

Copy link
Contributor

@albinekb albinekb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!! Let's go

@leo leo changed the title [WIP] Updated dependencies to the latest version Updated dependencies to the latest version Aug 31, 2017
@leo leo removed the request for review from chabou August 31, 2017 13:20
@leo leo merged commit 0ee4fb6 into master Aug 31, 2017
@leo leo deleted the update-dependencies branch August 31, 2017 13:20
chabou added a commit that referenced this pull request Sep 3, 2017
* master: (62 commits)
  1.4.3
  Disable ia32 linux releases (#2164)
  Fixed writing composed characters (#2158)
  Doc: Add yarn install to contribute instructions (#2117)
  Change "Close Session" shortcut on Linux/Windows  (#2160)
  Notice for plugins (#2114)
  Updated dependencies to the latest version (#2146)
  1.4.2
  Reverted class names to as they were before (#2139)
  1.4.1
  AppVeyor environment variables are now on the platform (#2137)
  Brought back the icon for closing tabs (#2136)
  Brought back keymap documentation to the website (#2133)
  1.4.0
  Don't build on master, except for releases (#2132)
  Ensured that `async-retry` is added to the bundle (#2131)
  Ensure correct update channel is displayed in About window (#2130)
  Retry loading it if config doesn't exist in auto updater (#2129)
  Write contents of default config to hyper.js (#2128)
  Use a string for setting the update channel (#2127)
  ...
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