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

chore: drop Node 6 #3598

Merged
merged 3 commits into from Jul 1, 2019
Merged

chore: drop Node 6 #3598

merged 3 commits into from Jul 1, 2019

Conversation

curbengh
Copy link
Contributor

@curbengh curbengh commented Jun 27, 2019

What does it do?

Drop Node 6 as per #3508 . It has reached EOL since 2019-04-30.

How to test

git clone -b drop-node6 https://github.com/weyusi/hexo.git
cd hexo
npm install
npm test

Pull request tasks

  • Passed the CI test.

@curbengh
Copy link
Contributor Author

curbengh commented Jun 27, 2019

Some considerations:

  • will there be v3.10.0?
    • it's still possible by creating a separate branch.
  • might need the following in .eslintrc if someone wants to use ES2017/2018.
  "parserOptions": {
    "ecmaVersion": 2018
  }

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.148% when pulling 75d23e2 on weyusi:drop-node6 into 7fba3c1 on hexojs:master.

@coveralls
Copy link

coveralls commented Jun 27, 2019

Coverage Status

Coverage remained the same at 97.148% when pulling 5a61477 on weyusi:drop-node6 into 7fba3c1 on hexojs:master.

@yoshinorin yoshinorin mentioned this pull request Jun 28, 2019
53 tasks
@yoshinorin
Copy link
Member

will there be v3.10.0?

I think maybe no.

it's still possible by creating a separate branch.

I prefer to create a branch by new version. But, hexo team seems has never created new branch. (Sorry I'm not familiar with hexo team development flow)

IMO, we can merge this PR to master. Because, maybe we will not release a patch version.

@yoshinorin
Copy link
Member

might need the following in .eslintrc in someone wants to use ES2017/2018.

I think maybe it's better to upgrade eslint-config-hexo.

@segayuu
How do you think?

@curbengh curbengh added this to the v4.0.0 milestone Jun 28, 2019
@curbengh
Copy link
Contributor Author

can someone restart the appveyor build with cleared cache?

@yoshinorin
Copy link
Member

How about re-commit & force-push?
Or, disable cache settings once by appveyor.yml

@segayuu
Copy link
Contributor

segayuu commented Jun 28, 2019

@yoshinorin eslint-config-hexo should be updated. It relies on eslint-config-node to properly manage the native API of node.js. Now that the LTS version of node.js has changed, it should be updated.

@curbengh
Copy link
Contributor Author

I created a new PR for eslint-config-hexo (hexojs/eslint-config-hexo#16).

@curbengh
Copy link
Contributor Author

How about re-commit & force-push?
Or, disable cache settings once by appveyor.yml

disable then re-enable cache works. Remember to squash them though.

Copy link
Member

@yoshinorin yoshinorin left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants