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.

Sign up for GitHub

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

Jump to bottom

Adds CI steps for atmosphere #2

Merged

mara-schulke merged 1 commit into helsing-ai:main from xfbs-fork:patrick/ci

Nov 25, 2023

Contributor

xfbs commented Nov 25, 2023

This adds some CI actions to this repository, copied from buffrs (because I suck at GitHub Actions):

Formatting (cargo fmt)

Fun fact: rustfmt nightly is currently slightly broken on aarch64 targets: rustfmt 1.7.0-nightly inconsistent output across targets rust-lang/rustfmt#5964.
Testing (cargo test, uses docker-compose file that I've placed into tests/postgres.yml)
Spelling (typos)
Clippy (cargo clippy)

I have not included the coverage check at this point, and also not the cargo-deny check (due to lack of a deny.toml).

Besides that:

Fixes some spelling errors, caught by typos (which is awesome btw, I did not know it before)
Some tests are failing and I don't know how to fix them yet (should not be too hard).


          Adds CI steps for atmosphere

0c01559

mara-schulke approved these changes

View reviewed changes

mara-schulke merged commit 7fa452d into helsing-ai:main

3 of 4 checks passed

Contributor

mara-schulke commented Nov 25, 2023

fixed them on main!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet