search

XAMPPRocky / octocrab

A modern, extensible GitHub API Client for Rust.
Other
1.07k stars 261 forks source link

cargo fmt #658

Closed maflcko closed 3 months ago

maflcko commented 3 months ago

Looks like this was forgotten before the latest merge commit.

XAMPPRocky commented 3 months ago

Thank you for your PR! Would you mind also adding to the CI check so we can prevent it happening in the future?

maflcko commented 3 months ago

I already added it to the CI :)

See CI run https://github.com/XAMPPRocky/octocrab/actions/runs/9484551402/job/26134293504 of commit f10ab52f7142410c456277a83eae7f1e77f3ba76 in the main branch.

The problem was that the CI didn't run on the last merged pull request. I think a setting exists in GitHub to prevent a merge before CI, but I haven't confirmed it.

XAMPPRocky commented 3 months ago

Ah right, gotcha. Thank you for your PR!