Munter / express-systemjs-translate

Express middleware to speed up systemjs development loads by running translations serverside
28 stars 4 forks source link

Update eslint to version 3.8.1 🚀 #205

Closed greenkeeperio-bot closed 7 years ago

greenkeeperio-bot commented 7 years ago

Hello lovely humans,

eslint just published its new version 3.8.1.

State Update :rocket:
Dependency eslint
New version 3.8.1
Type devDependency

This version is not covered by your current version range.

Without accepting this pull request your project will work just like it did before. There might be a bunch of new features, fixes and perf improvements that the maintainers worked on for you though.

I recommend you look into these changes and try to get onto the latest version of eslint. Given that you have a decent test suite, a passing build is a strong indicator that you can take advantage of these changes by merging the proposed change into your project. Otherwise this branch is a great starting point for you to work on the update.

Do you have any ideas how I could improve these pull requests? Did I report anything you think isn’t right? Are you unsure about how things are supposed to work?

There is a collection of frequently asked questions and while I’m just a bot, there is a group of people who are happy to teach me new things. Let them know.

Good luck with your project :sparkles:

You rock!

:palm_tree:


GitHub Release


The new version differs by 371 commits .

There are 250 commits in total. See the full diff.


This pull request was created by greenkeeper.io.

Tired of seeing this sponsor message? :zap: greenkeeper upgrade

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 84.393% when pulling 8594e44d365533c2943031b77122d9a3fcf0fee0 on greenkeeper-eslint-3.8.1 into f79c705ca39690c6a98c86d1573e1a3d4461e46f on master.

papandreou commented 7 years ago

Echoing a previous comment: @Munter @gustavnikolaj eslint 3 drops support for node.js 0.10 and 0.12, which raises the question -- are we OK with doing that, too? I wouldn't mind.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 84.393% when pulling 1c927a139df8ad0b01ac7649acc162c446b82c04 on greenkeeper-eslint-3.8.1 into 1467338ac326075422522a0cadad871ab501e59a on master.

Munter commented 7 years ago

Fine by me. I assume this will always be run on developer machines, where there is more control over the node version they want to run

papandreou commented 7 years ago

I agree!

Ditched node.js <4 in the Travis config, updated eslint, then went ahead and released 5.0.0.

Munter commented 7 years ago

👯 :dancing_men: