search

rust-embedded / linux-embedded-hal

Implementation of the `embedded-hal` traits for Linux devices
Apache License 2.0
236 stars 40 forks source link

Add github actions CI #57

Closed ryankurte closed 3 years ago

ryankurte commented 3 years ago

Replaces #56, just adding the actions executor without swapping bors or removing travis.

Odds are this doesn't work but, seemingly one can't add actions to a repo from a branch so it has to be manually merged anyway and fixed in PRs...

@posborne seem (un)reasonable?

rust-highfive commented 3 years ago

r? @posborne

(rust-highfive has picked a reviewer for you, use r? to override)

ryankurte commented 3 years ago

bors try

bors[bot] commented 3 years ago

try

Build succeeded:

eldruin commented 3 years ago

rustfmt seems to have failed. Could you fix that @ryankurte?

ryankurte commented 3 years ago

huh, looks like actions is now executing and i'm not sure what changed?

applied rustfmt, fingers crossed...

bors[bot] commented 3 years ago

Timed out.

eldruin commented 3 years ago

bors retry

bors[bot] commented 3 years ago

Timed out.

eldruin commented 3 years ago

bors retry

bors[bot] commented 3 years ago

Timed out.

eldruin commented 3 years ago

@ryankurte Since this works fine, could you also remove the .travis.yml file and update the bors.toml to have the correct names from GHA like here? I really can't wait to get rid of Travis....

ryankurte commented 3 years ago

bors try

bors[bot] commented 3 years ago

try

Build succeeded:

bors[bot] commented 3 years ago

Build succeeded:

eldruin commented 3 years ago

Finally! :tada: