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

Add link to action log if TF plan exceeds 65300 char #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add link to action log if TF plan exceeds 65300 char #25

wants to merge 1 commit into from

Conversation

vikram-katkar
Copy link

It would be a better idea to point to github actions logs if TF plan is big enough to not fit PR character limit.
Wdyt?

@vikram-katkar
Copy link
Author

pr-comment

@cayter
Copy link

cayter commented Jan 7, 2022

@robburger Any issue with the PR?

@allankp
Copy link

allankp commented Jan 25, 2022

@cayter Nice work, would be great to have this merged.

@AliHill0
Copy link

This would be really useful, can we merge this?

@dkooll
Copy link

dkooll commented Apr 13, 2022

@robburger would be great if this one can be merged

@sunnyoswalcro
Copy link

when can this be merged ?

@Office-Manager
Copy link

@vikram-katkar have you considered forking this since it appears it's not longer in active development.
Or have you found an alternative

@rdhar
Copy link

rdhar commented Aug 21, 2023

I'm looking for this use-case as well: some way of linking back to the workflow logs if the Terraform plan output is too long to add as a comment.

Found a solution DevSecTop/TF-via-PR repository, which hosts a reusable workflow to run Terraform commands via PR comments, like a CLI.

Here's an example of how the output PR comments made by the workflow links back to the triggering workflow logs, where the plan can be viewed in full or downloaded as an artifact.

juan-vg referenced this pull request in juan-vg/terraform-pr-commenter Jun 28, 2024
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.

8 participants