Continuous Integration #22
Conversation
Looks like it's an org admin (docs). Travis has some docs here, but I'm not sure if that's for the new GitHub App approach. Perhaps an org admin needs to go through the process starting here for the $0 open source plan? @carols10cents, could you look into setting this up at the org level? |
|
Moving the fetcher to a separate, unpublished crate makes sense to me, and seems orthogonal to CI. Can you spin it out into a separate PR? |
|
Sure |
|
|
||
| matrix: | ||
| include: | ||
| - os: osx |
There was a problem hiding this comment.
I think we want to include - os: linux as well.
There was a problem hiding this comment.
The default is linux, this adds an extra build that is run against macOS on top of those. You can see the full build matrix here: https://travis-ci.org/Nemo157/license-exprs/builds/376051360
There was a problem hiding this comment.
How it actually works, each entry in the include array defines one job, the first job here overrides the os to be osx and uses the default rust version, the other 3 jobs each override the rust version and use the default os of linux.
There was a problem hiding this comment.
This looks good to me. Unless someone weighs in with additional issues, I'll merge it once @carols10cents has time to enable Travis for the org.
|
I think I did it? https://travis-ci.com/rust-lang-nursery/license-exprs |
Yup, looks good to me. I've merged this branch, and the new master is running through its tests here. Maybe I should have kicked Travis somehow before merging to make sure everything passed, but it's too late now ;). |
|
I guess it's under travis-ci.com instead of travis-ci.org because it's using the new check application support? I wonder if all the other |
They have a migration tool, if folks want to do that in bulk. And without that, you can also opt-in to having Travis migrate you. |
Changes since v1.3.0: * Updated SPDX License List to 3.1 (ehuss#9, ehuss#11, ehuss#17, ehuss#19, ehuss#24) * Updated SPDX License Expression reference from 2.0 to 2.1 (ehuss#8). * Document our license-list version, parens issue, and license (ehuss#16, ehuss#17). * Add Travis CI configuration (ehuss#22). * Add additional test cases (ehuss#25). * .mailmap: Consolidate authors (ehuss#26).
Changes since v1.3.0: * Updated SPDX License List to 3.1 (ehuss#9, ehuss#11, ehuss#17, ehuss#19, ehuss#24). * Updated SPDX License Expression reference from 2.0 to 2.1 (ehuss#8). * Document our license-list version, parens issue, and license (ehuss#16, ehuss#17). * Add Travis CI configuration (ehuss#22). * Add additional test cases (ehuss#25). * .mailmap: Consolidate authors (ehuss#26).
I assume someone with permission would have to enable this repository in Travis CI, not sure what exact permission that would be though.