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

Test with multiple TypeScript minor versions in CI #217

Merged
merged 3 commits into from
Oct 16, 2019

Conversation

nickmccurdy
Copy link
Collaborator

@nickmccurdy nickmccurdy commented Oct 15, 2019

Fixes #216

  • Add build matrix to Travis for multiple TypeScript versions
  • Update development version to 3.6

@netlify
Copy link

netlify bot commented Oct 15, 2019

Deploy preview for redux-starter-kit-docs ready!

Built with commit ec24833

https://deploy-preview-217--redux-starter-kit-docs.netlify.com

@markerikson
Copy link
Collaborator

Seems pretty straightforward, and the Travis run for this PR definitely ran for each of those. Anything else needed to tweak this?

@nickmccurdy
Copy link
Collaborator Author

nickmccurdy commented Oct 16, 2019

Oops the after_install script that actually installs the right TypeScript version isn't working, I'll look into this

@nickmccurdy
Copy link
Collaborator Author

The multiple TypeScript versions are being installed properly now, but the type tests fail on TypeScript 3.6. Should we try to fix it with this PR or use 3.5 as the max for now?

@nickmccurdy nickmccurdy marked this pull request as ready for review October 16, 2019 03:28
@markerikson
Copy link
Collaborator

Merge down from master - I merged in #215 , which should hopefully fix that.

@markerikson
Copy link
Collaborator

Yep, that passes now. Should be good to go!

@markerikson markerikson merged commit 93d63f0 into master Oct 16, 2019
@markerikson markerikson deleted the multiple-ts-versions branch October 16, 2019 03:55
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.

Update Travis to test against multiple versions of TypeScript
2 participants