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
CI #1870
CI #1870
Conversation
CI is looking good 👍 I had to restart because of one of the Windows jobs though. I'm okay with merging as is, but I'd like to split out the CI jobs into multiple workflow files, unless there are any objections? |
CI is broken for Node 8 beacuse semantic-release won't install. This isn't just breaking all the builds, it's also blocking release. We're dealing with updating the CI this in #1870 and maybe we'll be able to fix this. If not, we should also prioritize #1871 so we don't accidentally ship a breaking change for Node 8. I'm removing this for now, so we can get the patch for #1836 shipped.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would we disable Travis completely after merging this?
Not yet, we'll still need to move semantic-release over to actions |
Looks like this work was merged via #1814, right? |
follow up to #1814 to test new CI jobs