-
Notifications
You must be signed in to change notification settings - Fork 42
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
More CI checks #93
More CI checks #93
Conversation
4fdfc59
to
99fbc4d
Compare
This is still a lot of changes all in one commit. Please split it up further, it can be in one PR but it should be reviewably split into multiple commits, that have descriptive commit messages when they're making nontrivial changes. Where are all the extra tests coming from? What's with all of the changes to the python script? Some seem to be python formatting (fine), others actually change the structure/behavior of the code? |
The changes to the Python script are to ensure that the Rust code files it outputs pass rustfmt. I didn't make any changes to how the normalization tests are generated; I just regenerated the file, which was outdated before (as not checked by CI). |
4899ec1
to
c2cb7cd
Compare
I've split the PR into four commits. |
@Jules-Bertholet okay, can you remove the Python changes? Let's have |
And add it to `Cargo.toml`
c2cb7cd
to
0b9b162
Compare
Done |
And also mass-reformat. `tables.rs` is ignored for now
0b9b162
to
b077494
Compare
CI fails
|
38c395a
to
261ca20
Compare
Splits out "unrelated" changes from #92 as requested.