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

Publish build results from Jenkins to GitHub #623

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

ptziegler
Copy link
Contributor

@ptziegler ptziegler commented Nov 18, 2024

Main goal is to analyze what steps need to be taken to have the Jenkins build results like compiler warnings and Maven problems be published to the GitHub checks.

@ptziegler
Copy link
Contributor Author

See #472 as a potential use-case

@ptziegler
Copy link
Contributor Author

Hm... The upload is done via the github-checks plugin, while the diff with the reference build is done by the git-forensics plugin. I assume both need to be installed in our Jenkins instance.

@ptziegler
Copy link
Contributor Author

@ptziegler ptziegler changed the title Investigate settings to publish build results fron Jenkins to GitHub Publish build results (e.g. compiler warnings) from Jenkins to GitHub Nov 19, 2024
@ptziegler ptziegler force-pushed the jenkins-reports branch 2 times, most recently from 3d46ba2 to 0dde7c5 Compare November 19, 2024 15:31
@ptziegler ptziegler changed the title Publish build results (e.g. compiler warnings) from Jenkins to GitHub Publish build results from Jenkins to GitHub Nov 19, 2024
This initial step is just about how to get the results (e.g. compiler
warnings) to GitHub. In a later stage, those can the be further
configured to have the build fail if it would introduce additional
warnings.
@ptziegler ptziegler marked this pull request as ready for review November 19, 2024 15:44
@ptziegler
Copy link
Contributor Author

@azoitl wdyt? For the next release, I'd like to make the quality checks a little harsher so that e.g. builds fail if they introduce new warnings. To make things a little bit more user-friendly, those warnings should therefore be visible on GitHub as well.

image

@azoitl
Copy link
Contributor

azoitl commented Nov 20, 2024

I find that a great idea!

@azoitl azoitl merged commit 99a9190 into eclipse-gef:master Nov 20, 2024
14 checks passed
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.

2 participants