add windows and osx to travis

[davidmarkclements]

No description provided.

[davidmarkclements]

note - we might want to wait on merging this - it looks like windows builds fail without any useful logs - windows on travis is in beta still

[trygve-lie]

Try to set sudo to true. I had simmilar issue and it turned out to be that Windows could not install dependencies without administrator rights. Yeah, logs are kinda useless on this.

[kibertoad]

Windows support in pino tests is broken altogether - I fixed it along the road in #626

[kibertoad]

Windows builds seem to just die quietly...

[davidmarkclements]

Windows builds seem to just die quietly...

yeah... annoying isn't it. Tried changing sudo: true - still failing

[kibertoad]

Well, they do claim that windows support is quite preliminary. Maybe just run on Mac & Linux for now?

[davidmarkclements]

well, the thing is I have windows successfully working on other projects – e.g. https://travis-ci.org/esxjs/esx

so we need to check if pino tests run on windows at all (last we checked they did) and work from there

[kibertoad]

@davidmarkclements Right, so here's the thing with tests on Windows - they are failing since we are skipping pretty.test.js tests on Windows due to colour code discrepancy, and this result in code coverage being below the threshold and thus task fails. Should the platform-specific assertion be implemented for those tests?

[jsumners]

This is covered by the move to GitHub Actions (which is complete in #747).

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.