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

Make sure that output.text is set in each checks call #1033

Merged
merged 1 commit into from
Feb 8, 2021

Conversation

uhafner
Copy link
Contributor

@uhafner uhafner commented Feb 4, 2021

Description

If a GitHub checks contains more than 50 annotations, then the
check is split into several calls. This fix makes sure that all
additional calls will not only copy the properties title and
summary but also the previously missing property text.

Before submitting a PR:

We love getting PRs, but we hate asking people for the same basic changes every time.

  • Push your changes to a branch other than master. Create your PR from that branch.
  • Add JavaDocs and other comments
  • Write tests that run and pass in CI. See CONTRIBUTING.md for details on how to capture snapshot data.
  • Run mvn clean compile locally. This may reformat your code, commit those changes.
  • Run mvn -D enable-ci clean install site locally. If this command doesn't succeed, your change will not pass CI.

When creating a PR:

  • Fill in the "Description" above.
  • Enable "Allow edits from maintainers".

If a GitHub checks contains more than 50 annotations, then the
check is split into several calls. This fix makes sure that all
additional calls will not only copy the properties `title` and
`summary` but also the previously missing property `text`.
@timja timja added the bug label Feb 4, 2021
@timja timja requested a review from bitwiseman February 4, 2021 19:58
@bitwiseman bitwiseman merged commit d2732bc into hub4j:master Feb 8, 2021
@uhafner
Copy link
Contributor Author

uhafner commented Feb 24, 2021

@bitwiseman It would be nice it you could cut a new release with that fix.

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

Successfully merging this pull request may close these issues.

3 participants