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.

Sign up for GitHub

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

Jump to bottom

Remove Travis CI tests #704

Closed
bgrainger opened this issue Sep 23, 2019 · 2 comments
Closed

Remove Travis CI tests #704

bgrainger opened this issue Sep 23, 2019 · 2 comments
Assignees
@bgrainger

Comments

@bgrainger
Copy link
Member

bgrainger commented Sep 23, 2019 •
edited
Loading

The Travis CI tests time out on PR builds (see #646 (comment) and #702) and are mostly duplicated by the Azure Pipelines tests, with the following exceptions:

  • netcoreapp1.1.2 integration tests
    • Can probably stop running these, since .NET Core 3.0 is now out. netstandard1.3 would be a supported but untested platform. (See Drop support for net461? #636 for possibly removing it.)
  • Compression integration tests
    • Are being run on Appveyor (for Windows); probably would be good to test on Azure Pipelines
  • Unix Domain Socket tests
    • We don't need to run the full suite of integration tests against Unix sockets (since 99% of the code is the same) but it would be good to add a simple test of opening the connection to the Linux test suites
@bgrainger
Copy link
Member Author

Travis CI builds are now disabled on PRs.

@bgrainger bgrainger self-assigned this Sep 24, 2019
@bgrainger
Copy link
Member Author

Running Unix domain socket tests here: 7b62c79

Running compression tests here: d8e2f78

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bgrainger bgrainger

Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bgrainger