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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants