Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

fix: Trim whitespace from key-value pairs in markdown tiles #868

Merged
merged 2 commits into from
Aug 3, 2022

Conversation

arthurpitman
Copy link
Collaborator

This PR:

  • trims whitespace from key-value pairs from markdown tiles

This prevents also issues with additional whitespace at the end of markdown tiles.
Signed-off-by: Arthur Pitman [email protected]

j-poecher
j-poecher previously approved these changes Aug 3, 2022
Copy link
Contributor

@j-poecher j-poecher left a comment

Choose a reason for hiding this comment

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

lgtm

@arthurpitman arthurpitman changed the base branch from master to release-0.23.x August 3, 2022 11:09
@arthurpitman arthurpitman dismissed j-poecher’s stale review August 3, 2022 11:09

The base branch was changed.

@arthurpitman arthurpitman marked this pull request as ready for review August 3, 2022 11:09
Signed-off-by: Arthur Pitman <[email protected]>
@sonarcloud
Copy link

sonarcloud bot commented Aug 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link
Contributor

github-actions bot commented Aug 3, 2022

The following Docker Images have been built:
keptncontrib/dynatrace-service:0.23.1-dev-PR-868,keptncontrib/dynatrace-service:0.23.1-dev-PR-868.202208031112

@arthurpitman arthurpitman changed the base branch from release-0.23.x to release/0.23.x August 3, 2022 11:23
Copy link
Contributor

@j-poecher j-poecher left a comment

Choose a reason for hiding this comment

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

lgtm

@j-poecher j-poecher merged commit 2449853 into release/0.23.x Aug 3, 2022
@j-poecher j-poecher deleted the fix/handle-white-space-in-markdown-tiles branch August 3, 2022 11:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants