-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Make terraform plan comment collapsible #181
Comments
Yes I totally agree. I'm closing this as a dupe of #181 though because it's basically the same issue and there's some good ideas there. |
nishkrishnan
pushed a commit
that referenced
this issue
Feb 9, 2022
* Add UUID for Log Streaming Job ID (#167) * Update log handler to close buffered channels when an operation is complete (#170) * Add preliminary check before registering new receivers in the log handler (#173) * Using projectOutputBuffers to check for jobID instead of receiverBuffers (#181) * Refactor log handler (#175) * Reverting go.mod and go.sum * Fix lint errors * Fix linting
jamengual
pushed a commit
that referenced
this issue
Nov 23, 2022
krrrr38
pushed a commit
to krrrr38/atlantis
that referenced
this issue
Dec 16, 2022
* Add UUID for Log Streaming Job ID (runatlantis#167) * Update log handler to close buffered channels when an operation is complete (runatlantis#170) * Add preliminary check before registering new receivers in the log handler (runatlantis#173) * Using projectOutputBuffers to check for jobID instead of receiverBuffers (runatlantis#181) * Refactor log handler (runatlantis#175) * Reverting go.mod and go.sum * Fix lint errors * Fix linting
meringu
pushed a commit
to meringu/atlantis
that referenced
this issue
May 29, 2023
check if err is nil before printing it
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
terraform plan
output can sometimes be quite long and for WIP PRs, the number of comments can make for a long PR.Could we make the
terraform plan
output formatted as a collapsed code block in Markdown?The text was updated successfully, but these errors were encountered: