Commit Graph

11 Commits

Author SHA1 Message Date
Goh Jia Hao 95fc46d38d Integrate prettier 2018-05-24 00:02:44 -07:00
Goh Jia Hao 949dcfadd8 Migrate Mocha tests to Jest 2018-05-23 23:44:03 -07:00
Ronan Jouchet 6a8114e505 Fix deprecations: electron-packager cb->promise, mocha compilers->require 2018-05-11 16:55:50 -04:00
Matt Harris eb08d85830 Fix #480 - Move all console.* to loglevel.* calls, eslint-fail on console.* (PR #507) 2017-12-19 08:42:06 -05:00
Ronan Jouchet 6fb3b92eb8
Upgrade dependencies and default to latest Electron 1.7.9 (PR #483)
* Update deps except eslint
* Update eslint and lint:fix (WIP, needs manual fixing for remaining 44 problems)
* Manually fix remaining eslint errors
* Document deprecation of `version-string` as of electron-packager 9.0.0
* Upgrade to Electron 1.7.9 (chrome-58, node-7.9.0, v8-5.8)
* npm: Disable generation of package-lock.json and gitignore it
  --Trying this, package-lock is a pain in PRs. May not be a good idea
  (obviously we lose deps pinning), will revert if necessary.--
* npm tasks: add dev-up-win for Windows developers,
  and e2e for end-to-end tests. Update docs.
* Move normalizeUrl test to a jest unit test, makes no sense to be in the mocha e2e tests
* Switch from babel-preset-es2015 to babel-preset-env,
  with target.node=4.0. Seem like it's today's most convenient
  way to support the latest ES and let babel transpile to what
  makes sense for our currently minimal node version
2017-11-14 08:05:01 -05:00
Jia Hao Goh 8f78dd03af Update eslint and use Airbnb style
- Add `npm run lint:fix` command
- Cleanup inferIcon.js logic slightly
2017-04-29 22:52:12 +08:00
Darren Haken 461c7a38f0 Change Mocha to not need a babel build to run (#349)
* Change Mocha to not need a babel build to run

- Also add tests around normalizeUrl

* PR 359 Apply changes due to comments

- Remove babelrc as its in the package.json
- Change tdd npm task to use gulp
- Remove source map support file from import list for normalizeUrlSpec
- Change gulp tdd task to run mocha on first run
 359 Apply changes due to comments

 - Remove babelrc as its in the package.json
 - Change tdd npm task to use gulp
 - Remove source map support file from import list for normalizeUrlSpec
 - Change gulp tdd task to run mocha on first run
 359 Apply changes due to comments

 - Remove babelrc as its in the package.json
 - Change tdd npm task to use gulp
 - Remove source map support file from import list for normalizeUrlSpec
 - Change gulp tdd task to run mocha on first run
2017-04-25 16:04:57 +01:00
Goh Jia Hao 6fccbbfaa5 Use original eslint module for linting instead of gulp
- Workaround for google/eslint-config-google#10
2016-05-27 02:23:37 +08:00
Jia Hao aeddcea4cb No need to run CI test for gulp release
CI tests should be run before starting the release process
2016-03-14 12:44:30 +08:00
Jia Hao e0e5b16f0b DRY for building es6 with babel 2016-02-28 01:07:52 +08:00
Jia Hao 99fcb60a44 Split gulpfile into multiple parts 2016-02-28 00:45:44 +08:00