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.

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

Migrate from Travis CI to GitHub Actions #220

Merged
merged 19 commits into from
Jun 8, 2022
Merged

Migrate from Travis CI to GitHub Actions #220

merged 19 commits into from
Jun 8, 2022

Conversation

saschanaz
Copy link
Collaborator

@saschanaz saschanaz commented Jun 7, 2022

Closes #213, closes #218

@saschanaz
Copy link
Collaborator Author

actions/checkout@v3, actions-rs/toolchain@v1, and actions-rs/cargo@v1 are not allowed to be used in mozilla/rkv. Actions in this workflow must be: within a repository that belongs to your Enterprise account.

Boo.

@saschanaz
Copy link
Collaborator Author

https://bugzilla.mozilla.org/show_bug.cgi?id=1730873 says we don't want actions-rs. action-rs/cargo is not strictly needed, but not sure how to ditch action-rs/toolchain cleanly.

@saschanaz saschanaz closed this Jun 7, 2022
@saschanaz saschanaz reopened this Jun 7, 2022
@saschanaz saschanaz marked this pull request as ready for review June 7, 2022 21:42
@saschanaz saschanaz requested a review from badboy June 7, 2022 21:42
@@ -30,7 +30,7 @@ with-fuzzer-no-link = ["lmdb-rkv/with-fuzzer-no-link"]
[dependencies]
arrayref = "0.3"
bincode = "1.0"
bitflags = "1.1"
bitflags = "~1.2"
Copy link
Collaborator Author

@saschanaz saschanaz Jun 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.3 causes build failure on Rust 1.45.0. Maybe okay to bump the minimum supported version, but not here.

(Not sure 1.45.0 is even intended to be minimum supported version, there's no documentation for that 🤔)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MSRV can probably be lifted, we just need to stay in sync with what m-c requires: https://firefox-source-docs.mozilla.org/writing-rust-code/update-policy.html

# that we depend upon. When updating, pick a suitable nightly version
# from https://rust-lang.github.io/rustup-components-history/
# See .rustfmt.toml for the list of unstable features.
KNOWN_TO_HAVE_RUSTFMT: nightly-2020-07-12
Copy link
Collaborator Author

@saschanaz saschanaz Jun 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pinning to the existing old version from .travis.yml for now since newer version causes quite a lot of dead code warnings which I don't feel comfortable to fix here. (#221 for the trivial ones)

@@ -30,7 +30,7 @@ with-fuzzer-no-link = ["lmdb-rkv/with-fuzzer-no-link"]
[dependencies]
arrayref = "0.3"
bincode = "1.0"
bitflags = "1.1"
bitflags = "~1.2"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MSRV can probably be lifted, we just need to stay in sync with what m-c requires: https://firefox-source-docs.mozilla.org/writing-rust-code/update-policy.html

@saschanaz saschanaz merged commit 5fbe85f into main Jun 8, 2022
@saschanaz saschanaz deleted the gha branch June 8, 2022 11:14
@saschanaz saschanaz mentioned this pull request Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Travis CI free usage ends Dec 3; mozilla repos should switch to other CI platforms
2 participants