Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: plan too long for status api #18

Merged
merged 2 commits into from
Feb 18, 2021
Merged

fix: plan too long for status api #18

merged 2 commits into from
Feb 18, 2021

Conversation

edwinf
Copy link
Contributor

@edwinf edwinf commented Feb 18, 2021

Truncates text at 65400 characters and adds the ability to push entire output to a directory

Truncates text at 65400 characters and adds the ability to push entire output to a directory
@edwinf edwinf requested a review from williamwilson February 18, 2021 15:13
@@ -40,6 +40,7 @@ jobs:
token: ${{ secrets.GITHUB_TOKEN }}
working-directory: ${{ env.TERRAFORM_WORKING_DIRECTORY }}
report-title: Single Argument Run

Copy link
Contributor

Choose a reason for hiding this comment

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

nit: extra empty line

src/github.ts Outdated
}

function findLine(wording: string, output: string): string {
const wordingIndex = output.indexOf('Plan:');
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this be using the wording parameter?

@edwinf edwinf merged commit 1a78e7b into main Feb 18, 2021
@edwinf edwinf deleted the truncate-message branch February 18, 2021 17:04
github-actions bot pushed a commit that referenced this pull request Feb 18, 2021
## [1.1.7](v1.1.6...v1.1.7) (2021-02-18)

### Bug Fixes

* plan too long for status api ([#18](#18)) ([1a78e7b](1a78e7b))
@github-actions
Copy link

🎉 This PR is included in version 1.1.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants