CI

[bunny-therapist]

Github CI which runs tests, linting, formatting.

• Github actions

[xamgore]

As a temporary solution, I have set up git hooks running on git commit 👀

[bunny-therapist]

Git hooks are good, but they are up to each developer and can't (and shouldn't) replace CI :)

[bunny-therapist]

I have set up brain hooks for the same thing (my brain reminds me to run tests and formatting before committing).

[xamgore]

Git hooks are good, but they are up to each developer and can't (and shouldn't) replace CI :)

I mean, they are a dependency at Cargo.toml, rusty-hook. Which means the hooks are shared amongst developers.

[bunny-therapist]

They are optional but CI is mandatory.

[bunny-therapist]

Huh. rusty-hooks is automatic when using cargo? I did not know that. Similar features in other languages require special installation.

(We should probably still have CI though.)

[bunny-therapist]

Ah, not they do not run automatically? It is just that dev-requirements were installed automatically?

[xamgore]

The build stage of the dependency must be called. cargo check should do the job.

I usually set it up on git init, so that each developer has it. But here is an inherited repository, so... 😬

[bunny-therapist]

Is it possible to make the hooks optional once we have CI? They are currently driving me mad. For example, I can't stage and push a single file without first staging the rest of my changes, because it must all compile together, even though the commit just has that file. I always run checks before I push anything, so the hooks just limit what I can do.

[bunny-therapist]

Or can I just run git with "--no-verify" to skip the hooks? That should work, right?

[xamgore]

--no-verify is fine, yeah. You can turn it off in Intellij too, there are checkboxes at commit & push dialogs.

[xamgore]

We can relax hooks, like, move them to pre-push.