-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Move to GitHub Actions (instead of travis-ci.org) #1058
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
Comments
@stephencelis It's an important point, it takes about 5 minutes, and you are the only one having the permission to do it. |
@jberkel I talked with Stephen today, and we concluded we had to move to GitHub Actions. We still have an unresolved error with Swift Package Manager test command |
@nathanfallet thanks, I'll take a look. I saw that you merged some PRs, wouldn't it be better to sort out the CI before merging more code? |
@jberkel I only merged PRs about documentation (no CI needed) and little additions like conforming types (e.g. UUID or Int64) to codable things. I checked, and all tests are passing (except the SPM one, but others driven by Cocoapods or Carthage did), but yes, for other PRs we need to fix the CI first. |
Could the SPM build error be related to #1051? |
Yes, I found the problem will open a PR soon. Some of the last merged PRs broke the tests |
@jberkel Okay, waiting for the fix (also mark you PR to close this issue) |
The old travis.org service is no longer available.
You need to activate something in order to move the CI to travis-ci.com. PRs won't be able to get merged otherwise
The text was updated successfully, but these errors were encountered: