I would want to run the github action checks before merging, I think the github templates should be updated to trigger on pull_request instead of on push. It would be appreciated if you could update it so that we can test if it actually triggers for pull requests from public forks.

Otherwise great work and thank you for your contributions 😃

Done

I also noticed a bunch of clippy warnings about inlining format arguments. I can send in another PR after this one to fix them if you'd like @nathanielsimard

I also noticed a bunch of clippy warnings about inlining format arguments. I can send in another PR after this one to fix them if you'd like @nathanielsimard

Yes why not, but the CI is supposed to fail when there are clippy warnings. What command did you run?

I also noticed a bunch of clippy warnings about inlining format arguments. I can send in another PR after this one to fix them if you'd like @nathanielsimard

Yes why not, but the CI is supposed to fail when there are clippy warnings. What command did you run?

I'm actually on nightly rust, so the warnings may be new.
It's just cargo clippy

I've actually already fixed all the warnings except this

At