Skip to content

[#1931] Use ANSI escape codes in iceoryx2-cli - #1933

Open
dkroenke wants to merge 8 commits into
eclipse-iceoryx:mainfrom
ekxide:iox-1931-use-ansi-escape-code-in-cli
Open

[#1931] Use ANSI escape codes in iceoryx2-cli#1933
dkroenke wants to merge 8 commits into
eclipse-iceoryx:mainfrom
ekxide:iox-1931-use-ansi-escape-code-in-cli

Conversation

@dkroenke

Copy link
Copy Markdown
Contributor

Notes for Reviewer

  • Use ANSI Escape sequences in iceoryx2-cli
  • remove obsolete iceoryx2-cli/iox2-config/src/commands.rs as it is a duplicate of iceoryx2-cli/iox2-config/src/command/mod.rs
  • remove obsolete dirs crate in iceoryx2-cli

Pre-Review Checklist for the PR Author

  • Add sensible notes for the reviewer
  • PR title is short, expressive and meaningful
  • Consider switching the PR to a draft (Convert to draft)
    • as draft PR, the CI will be skipped for pushes
  • Relevant issues are linked in the References section
  • Branch follows the naming format (iox2-123-introduce-posix-ipc-example)
  • Commits messages are according to this guideline
    • Commit messages have the issue ID ([#123] Add posix ipc example)
    • Keep in mind to use the same email that was used to sign the Eclipse Contributor Agreement
  • Tests follow the best practice for testing
  • Changelog updated in the unreleased section including API breaking changes

PR Reviewer Reminders

  • Commits are properly organized and messages are according to the guideline
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

References

Relates #1931

@dkroenke
dkroenke requested a review from elfenpiff August 21, 2026 08:05
@dkroenke dkroenke self-assigned this Aug 21, 2026
elBoberido
elBoberido previously approved these changes Aug 21, 2026
Comment thread iceoryx2-bb/print/src/lib.rs
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.33%. Comparing base (b9a46eb) to head (49178c1).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1933   +/-   ##
=======================================
  Coverage   76.33%   76.33%           
=======================================
  Files         457      457           
  Lines       46068    46068           
  Branches     1489     1489           
=======================================
  Hits        35165    35165           
  Misses       9641     9641           
  Partials     1262     1262           
Flag Coverage Δ
CPP 62.61% <ø> (ø)
Rust 76.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 9 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants