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

Upgrade lab to its latest version #79

Closed
wants to merge 2 commits into from

Conversation

orditeck
Copy link

@orditeck orditeck commented Oct 25, 2018

  • Upgrade lab to its latest version (17.x.x) to get tests working
  • Refactor tests to make them work with the new lab
  • Now using async/await (100% sure it could be better), at least for before/after on describe('bin')
  • Now requires node >= 7.6.0 (because of async/await)
  • Bump version to 4.0.0 as there's a breaking change regarding node's version
  • Add a note on the readme to reflect the new node's version

.travis.yml Outdated
- "4"
- "6"
- "node"
- "7.6.0"
Copy link
Member

Choose a reason for hiding this comment

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

How do we feel about following hapi to test 8, 10, and latest?

https://github.com/hapijs/hapi/blob/master/.travis.yml

Choose a reason for hiding this comment

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

I think these versions are still very popular.

Copy link
Author

Choose a reason for hiding this comment

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

Saw that. Was wondering too. It'd probably make more sense. I'll copy them.

Copy link
Member

Choose a reason for hiding this comment

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

(Both v7 and v9 are already EOL: https://github.com/nodejs/Release)

Copy link
Author

Choose a reason for hiding this comment

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

Ah well! That would be why then, haha!

This was referenced Nov 3, 2018
@augnin
Copy link
Member

augnin commented Nov 7, 2018

closing in favour of #80 . Thanks for your effort. Feel free to submit another PR if I missed something in #80

@augnin augnin closed this Nov 7, 2018
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.

None yet

4 participants