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

fix: Resolve Rendering Issue in the Policy Check Results Template #4870

Merged

Conversation

dorian-tsorfy
Copy link
Contributor

what

I added a space line to the policy_check_results_wrapped.tmpl file so the output on GitLab will look nice.
Resolving issue #4623.

why

We have noticed intermittent issues rendering the markdown Atlantis outputs to GitLab when a policy check failure occurs ever since upgrading from v0.27.3 to v0.28.1. I suspect this could be related to #4254 and the formatting changes it made.

tests

references

#4623
#4787

@dorian-tsorfy dorian-tsorfy requested review from a team as code owners August 26, 2024 14:28
@dorian-tsorfy dorian-tsorfy requested review from GenPage, lukemassa and nitrocode and removed request for a team August 26, 2024 14:28
@github-actions github-actions bot added the go Pull requests that update Go code label Aug 26, 2024
@dorian-tsorfy
Copy link
Contributor Author

@X-Guardian,
Uploaded here a new PR for the space line

@dorian-tsorfy
Copy link
Contributor Author

@jamengual Hi
I created the PR again of adding a space line

@X-Guardian
Copy link
Contributor

Thanks @dorian-tsorfy, can you check this comment to see whether further fixes are needed: #4623 (comment)

@dorian-tsorfy
Copy link
Contributor Author

Thanks @dorian-tsorfy, can you check this comment to see whether further fixes are needed: #4623 (comment)

Done

@dorian-tsorfy
Copy link
Contributor Author

@GenPage @jamengual
Hey guys,
I saw my PR is included in the latest release v0.29.0 although it was reverted (check #4787).
I then created this PR that passes all the tests and is waiting to be merged.

@X-Guardian X-Guardian changed the title fix: added a space line fix: Resolve Rendering Issue in the Policy Check Results Template Sep 3, 2024
@X-Guardian X-Guardian merged commit 41cd9c2 into runatlantis:main Sep 3, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants