Ci improvements

[muraca]

• renamed build.yml to ci.yml
• use toml-sort action, removed toml-sort.yml
• removed check
• test with nextest, llvm cov for coverage ~ fixes #106
• added --no-deps to clippy
• bumped version for updated actions
• removed unmantained actions
• ⚠️ @JoshOrndorff ⚠️ tests and clippy start only after rustfmt and toml sort are successful, this is intentional and I would like to keep it

[JoshOrndorff]

Overall looks good. I left a few questions but I trust your judgement.

[muraca]

@JoshOrndorff please set Workflow permissions to read and write in Settings > Actions > General.

[JoshOrndorff]

For some reason, that option is disabled for me.

EDIT: Okay, I did it at the organization level and restarted the job.

According to https://docs.github.com/en/actions/security-guides/automatic-token-authentication#example-1-passing-the-github_token-as-an-input it looks like how you did pull_requests: write should work.

[muraca]

The problem is that the action is performed on my fork, and the bot is trying to comment on this repo using the secret key of my personal repo. This does not happen if the branch is part of this repo, see #112 or muraca#2 . Looks like there's nothing we can do to avoid it, so I'll try to run the step only if the repo of the head and the base are the same, otherwise print it in the action output

[muraca]

@JoshOrndorff please take a look at this https://github.com/Off-Narrative-Labs/Tuxedo/actions/runs/6411679096/job/17407615977?pr=110

[JoshOrndorff]

So this is the one you want to merge, right?

Go ahead and merge the one you when you're ready, and close the other one.

Is it possible to print the results to terminal in both cases? (And also comment when possible?) If not no big deal.

[muraca]

Is it possible to print the results to terminal in both cases? (And also comment when possible?)

yes, I am going to test it out right now.

[muraca]

#112