Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: Fix styling bug for target labels with special names #3692

Merged
merged 3 commits into from
Apr 4, 2021

Conversation

Namanl2001
Copy link
Contributor

Signed-off-by: Namanl2001 [email protected]

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Fixes a subissue Fix styling bug for target labels with special names. in #3541

onprem
onprem previously approved these changes Jan 21, 2021
Copy link
Member

@onprem onprem left a comment

Choose a reason for hiding this comment

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

LGTM. Let's rebase and then merge.

@kakkoyun kakkoyun enabled auto-merge (squash) February 10, 2021 11:05
@kakkoyun
Copy link
Member

@Namanl2001 If you could rebase it we can merge this PR.

Base automatically changed from master to main February 26, 2021 16:30
bwplotka
bwplotka previously approved these changes Apr 1, 2021
auto-merge was automatically disabled April 3, 2021 15:26

Head branch was pushed to by a user without write access

@Namanl2001 Namanl2001 dismissed stale reviews from bwplotka and onprem via 0b1d10b April 3, 2021 15:26
Signed-off-by: Namanl2001 <[email protected]>
@Namanl2001
Copy link
Contributor Author

Namanl2001 commented Apr 3, 2021

Resolved the conflicts. The failing test seems unrelated.

Copy link
Member

@onprem onprem left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@onprem onprem changed the title Fix styling bug for target labels with special names UI: Fix styling bug for target labels with special names Apr 4, 2021
@onprem onprem enabled auto-merge (squash) April 4, 2021 14:26
@onprem onprem merged commit f0702ed into thanos-io:main Apr 4, 2021
@Namanl2001 Namanl2001 deleted the fix-target-label-styling branch April 16, 2021 13:23
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.

4 participants