Skip to content

refactor(framework): Tweak flwr federation ls --federation view#6726

Merged
danieljanes merged 3 commits intomainfrom
tweak-fed-ls-view
Mar 9, 2026
Merged

refactor(framework): Tweak flwr federation ls --federation view#6726
danieljanes merged 3 commits intomainfrom
tweak-fed-ls-view

Conversation

@panh99
Copy link
Contributor

@panh99 panh99 commented Mar 9, 2026

Avoid the log message being auto highlighed.
Use normal color for federation members instead of dimmed color.

Before (active & archived)

image image

After (active & archived)

image image

Copilot AI review requested due to automatic review settings March 9, 2026 19:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts the flwr federation ls --federation CLI output to improve Rich color rendering and readability when showing a specific federation’s details.

Changes:

  • Removes the space before the trailing ellipsis in the “Showing … federation …” line to avoid unintended yellow highlighting.
  • Updates the federation members table to use normal (non-dimmed) styling for member entries.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@danieljanes danieljanes enabled auto-merge (squash) March 9, 2026 20:05
@github-actions github-actions bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Mar 9, 2026
@danieljanes danieljanes merged commit 3a462e7 into main Mar 9, 2026
70 checks passed
@danieljanes danieljanes deleted the tweak-fed-ls-view branch March 9, 2026 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainer Used to determine what PRs (mainly) come from Flower maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants