schaefera/notif-picker
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix rustfmt formatting in icon.rs #1

Merged
claude-bot merged 4 commits from fix/icon-rs-fmt into master 2026-07-29 13:58:01 +00:00
Conversation 0 Commits 4 Files changed 2 +42 -4
claude-bot commented 2026-07-29 10:14:24 +00:00
Collaborator
Copy link

Reorders imports and reflows a long binding in src/icon.rs so cargo fmt --check passes, which the new .forgejo/workflows/ci.yml enforces.

Test plan

  • cargo fmt --check passes
Reorders imports and reflows a long binding in `src/icon.rs` so `cargo fmt --check` passes, which the new `.forgejo/workflows/ci.yml` enforces. ## Test plan - [x] `cargo fmt --check` passes
claude-bot added 2 commits 2026-07-29 10:14:25 +00:00 
Runs cargo fmt/clippy/test/audit via the existing cargo-make ci task on every push and pull request.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Fix rustfmt formatting in icon.rs
Some checks failed
CI / ci (push) Failing after 42s
Details
CI / ci (pull_request) Failing after 5s
Details
d1fd272fbb 
Reorders imports and reflows a long binding so the new CI's
format-check job passes.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
schaefera added 1 commit 2026-07-29 10:25:12 +00:00
Fix CI job: run on default node image instead of rust:latest
Some checks failed
CI / ci (push) Failing after 20m50s
Details
CI / ci (pull_request) Failing after 1h20m38s
Details
4e65ef3843 
The rust:latest container has no Node.js, but actions/checkout@v4 and
actions/cache@v4 are JS actions that require it to execute at all,
so the job failed before any cargo/fmt steps ran. Use the default
docker runner image and install the Rust toolchain via rustup instead.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
schaefera added 1 commit 2026-07-29 10:29:15 +00:00
Only trigger CI push runs on master to avoid duplicate builds
All checks were successful
CI / ci (pull_request) Successful in 10m41s
Details
1cda8b6f50 
Pushing to a branch with an open PR fired both the push and
pull_request triggers for the same commit. pull_request already
covers feature-branch commits, so push only needs to cover master.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
schaefera approved these changes 2026-07-29 13:57:16 +00:00
claude-bot merged commit bfef9f3fd6 into master 2026-07-29 13:58:01 +00:00
claude-bot deleted branch fix/icon-rs-fmt 2026-07-29 13:58:01 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
schaefera
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: schaefera/notif-picker#1
No description provided.
Delete branch "fix/icon-rs-fmt"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?