Skip to content

Conversation

@Techassi
Copy link
Member

This PR adds a Display impl for ClusterCondition. It might be worth a thought that we want to provide different display_* methods for various levels of verbosity.

@Techassi Techassi self-assigned this Jun 16, 2023
@Techassi Techassi marked this pull request as ready for review June 20, 2023 12:53
@Techassi Techassi requested a review from a team June 20, 2023 12:54
@Techassi Techassi requested a review from sbernauer June 20, 2023 14:08
@Techassi Techassi requested a review from sbernauer June 21, 2023 08:15
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Thx!

@Techassi
Copy link
Member Author

bors r+

bors bot pushed a commit that referenced this pull request Jun 21, 2023
This PR adds a `Display` impl for `ClusterCondition`. It might be worth a thought that we want to provide different `display_*` methods for various levels of verbosity. Co-authored-by: Techassi <sascha.lautenschlaeger@stackable.tech>
@bors
Copy link
Contributor

bors bot commented Jun 21, 2023

Pull request successfully merged into main.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

  • All tests passed
@bors bors bot changed the title feat: add Display for ClusterCondition [Merged by Bors] - feat: add Display for ClusterCondition Jun 21, 2023
@bors bors bot closed this Jun 21, 2023
@bors bors bot deleted the feature/cluster-conditions-display branch June 21, 2023 09:51
bors bot pushed a commit that referenced this pull request Jun 21, 2023
The display string for the `ReconciliationPaused` was swapped. This fixes the display impl introduced in #608. Co-authored-by: Techassi <sascha.lautenschlaeger@stackable.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants