Skip to content

Commit

Permalink
add more standard version
Browse files Browse the repository at this point in the history
  • Loading branch information
@kjetilk
kjetilk committed Oct 24, 2018
1 parent 12126e1 commit 6426f68
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .travis.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@ sudo: false
language: node_js

node_js:
- "8.10"
- "8.0"
- "10.0"
- "node"

cache:
directories:
Expand Down

2 comments on commit 6426f68

@smalinin
Copy link
Contributor

@smalinin smalinin commented on 6426f68 Oct 25, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

@kjetilk Travis reports error for execute tests with node 8.0 => Travis test 

/home/travis/build/solid/oidc-rp/test/AuthenticationResponseSpec.js:176
      const response = new AuthenticationResponse({ params: {...errorParams} })
                                                             ^^^

It looks, that such Spread Syntax isn't supported by Node 8.0 else.
It must be supported from Node 8.3.0 based on reference => Spread Syntax reference

@kjetilk
Copy link
Member Author

Choose a reason for hiding this comment

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

Uh, OK, thanks!

Please sign in to comment.