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
Action fails to decorate PR when text is too long #730
Labels
Comments
@henriquevcosta thanks for the report. Can you post the configuration you're using for the Action? |
@febuiles Please see below. This is in a reusable workflow called from elsewhere, I've left the value of those inputs in comments.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I just ran this action in a repo where the action failed with the error below and no content at all was posted to the workflow.
Would it be possible, in these cases, to leave a PR comment with the semantic of "The dependency review found new vulnerabilities in this change but was unable to display them here. You can see them in the output of the build here: <link to the action output/logs>" ? I know that a failed Check already has some of this semantic, but creating a comment provides a place to centralize discussion and clarity to developers.
Additionally, of course, maybe consider reviewing the output format to get more compact output in these cases by removing the "Scanned manifest files" section or trimming vulnerability descriptions to a fixed number of characters?
The text was updated successfully, but these errors were encountered: