search

graalvm / setup-graalvm

GitHub Action for setting up GraalVM distributions.
https://www.graalvm.org
Universal Permissive License v1.0
195 stars 28 forks source link

Add input option to update pr reports #102

Closed Mrflatt closed 3 months ago

Mrflatt commented 3 months ago

Adds native-image-pr-reports-update option, which replaces old native-image pr report comment with new one. This makes pr's less verbose.

Related to open issue: https://github.com/graalvm/setup-graalvm/issues/101

oracle-contributor-agreement[bot] commented 3 months ago

Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA). The following contributors of this PR have not signed the OCA:

To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application.

When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated.

If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public.

oracle-contributor-agreement[bot] commented 3 months ago

Thank you for signing the OCA.

Mrflatt commented 3 months ago

Thanks a lot for this contribution! I left a couple of comments and I hope they are easy to address (otherwise let me know). Also, it'd be great if you could sign the OCA so that we can integrate this.

Thanks again, Fabio

Thank you for review! Made changes according to comments

fniephaus commented 3 months ago

I've decided to rename the option to native-image-pr-reports-update-existing, which hopefully makes it a bit clearer.