Skip to content

feat: fix duplicate execution start banner emission#728

Merged
Timi16 merged 1 commit intoTimi16:mainfrom
blessme247:feature/deduplicate-execution-start
Mar 29, 2026
Merged

feat: fix duplicate execution start banner emission#728
Timi16 merged 1 commit intoTimi16:mainfrom
blessme247:feature/deduplicate-execution-start

Conversation

@blessme247
Copy link
Copy Markdown
Contributor

Description

This PR fix duplicate execution start banner emission in src/cli/command.rs

Related Issue

Closes #696

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • [ x] Refactor / code cleanup
  • Performance improvement
  • Test / CI improvement

CI/Test Behavior Changes

This PR does not modify any existing tests. However, some syntax error in were fixed in src/plugin/loader.rs to resolve build issues.

What changed in CI or test behavior?
N/A — no CI/test behavior changes

Migration notes

N/A


Checklist

  • All tests pass locally (cargo test --workspace --all-features)
  • Code is formatted (cargo fmt --all -- --check)
  • Clippy is clean (cargo clippy --workspace --all-targets --all-features -- -D warnings)
  • [ x] Commit message follows Conventional Commits
  • [x ] PR description mentions the related issue(s)
  • CI/test behavior changes documented above (or marked N/A)
  • If CLI flags/subcommands/help text changed, man pages regenerated (make regen-man) and .1 files committed

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 29, 2026

@blessme247 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Timi16 Timi16 merged commit cdf39cc into Timi16:main Mar 29, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

run output: de-duplicate execution start banner emission

2 participants