Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Enable github actions #25

Closed
wants to merge 2 commits into from
Closed

Enable github actions #25

wants to merge 2 commits into from

Conversation

rillian
Copy link

@rillian rillian commented Dec 15, 2020

Some further cleanup I did on the master branch after submitting #24, in case you'd like to do something similar.

  • Remove custom lints, since they don't seem portable.
    • Fixes cargo +nightly bench
  • Starter github actions script, since travis is dropping their free tier.
    • Default branch only; might need amending since you use dev branches.
  • Opt into rust 2018 edition for better future-proofing.
  • Bump rand to reduce the dep count.

Please let me know if you'd like me to redo this against another branch.

@rillian rillian changed the title Furthetr cleanup: Rust 2018, current nightly, github actions Further cleanup: Rust 2018, current nightly, github actions Dec 15, 2020
@Razican
Copy link
Owner

Razican commented Dec 16, 2020

Thanks for the PR, most of these changes will be available as soon as I release the 2.1.0 version, which I expect to do before the end of the year, as soon as all tests are passing :)

The only thing I would like to set up would be GitHub actions and using master as the default branch, which I think I will leave it for Q1 next year, since I don't have a lot of free time to do it now, and having an up-to-date version release should be a priority compared to having the CI "right".

@rillian
Copy link
Author

rillian commented Dec 17, 2020

No worries. I was just curious about the implementation and found I couldn't build it. :)

I wouldn't say working CI is lower priority than a release though. Continuous integration feedback is a semi-external review of the software's status. It gives both authors and users confidence that the code is in a good state during development and when releases are made.

Thanks for publishing the crate!

@Razican
Copy link
Owner

Razican commented Dec 17, 2020

No worries. I was just curious about the implementation and found I couldn't build it. :)

I wouldn't say working CI is lower priority than a release though. Continuous integration feedback is a semi-external review of the software's status. It gives both authors and users confidence that the code is in a good state during development and when releases are made.

Yes, but for now, the CI is working, as you can see here. It needs an improvement, though, since it's using a setup that was nice 4 years ago, but now things have changed :)

Thanks for publishing the crate!

I'm happy is helpful!

Copy the rust starter workflow for github actions. This does
a basic build and test with the default toolchain so we get
some continuous integration feedback for work on that platform.
Link the continuous integration results page from the README,
which is displayed on the landing page for many repository
hosting services and on the crates.io and lib.rs sites.

With travis dropping most free support at the end of 2020,
this will help maintain visible ci coverage.
@rillian rillian changed the title Further cleanup: Rust 2018, current nightly, github actions Enable github actions Dec 17, 2020
@rillian
Copy link
Author

rillian commented Dec 17, 2020

Ok, I've updated the PR to just add the basic github actions config, since you've addressed the other issues in your own way. Although I still think the custom lint levels will prove to be brittle.

Thanks for unbreaking the default branch! Is there an analysis available of the test failures on Windows?

@Razican
Copy link
Owner

Razican commented Dec 21, 2020

Ok, I've updated the PR to just add the basic github actions config, since you've addressed the other issues in your own way. Although I still think the custom lint levels will prove to be brittle.

Thanks :) I will give it a look when I have a bit of free time.

Thanks for unbreaking the default branch! Is there an analysis available of the test failures on Windows?

Test failures in Windows seem due to a small stack present in Travis-CI builds, but I'm not sure. I will check it again with GitHub actions.

@Razican Razican self-requested a review December 21, 2020 21:32
@Razican Razican deleted the branch Razican:master October 22, 2023 08:41
@Razican Razican closed this Oct 22, 2023
@rillian
Copy link
Author

rillian commented Oct 22, 2023

Looks like this was obsoleted by #29.

@Razican
Copy link
Owner

Razican commented Oct 22, 2023

Looks like this was obsoleted by #29.

Oh! I thought this was forgotten. I'll review my PR and add your changes if still relevant.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants