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

GitHub actions for pull requests #831

Merged
merged 4 commits into from
Jul 26, 2024
Merged

Conversation

csware
Copy link
Contributor

@csware csware commented Jul 23, 2024

I prepared a few GitHub actions so that people who open a pull request (and you) get some comments about the code.

An example with check results can be seen here: csware#4 and inline check results on https://github.com/csware/htmlunit/pull/4/files

Btw. rights now it quite often failws because https://oss.sonatype.org/ is kinda unstable (it's slow and often results in a 502 Bad gateway. Maybe using the GitHub package repository may make sense.

Signed-off-by: Sven Strickroth <email@cs-ware.de>
Signed-off-by: Sven Strickroth <email@cs-ware.de>
This required to adjust CodeStyleTest to have a unique output format.

Signed-off-by: Sven Strickroth <email@cs-ware.de>
Signed-off-by: Sven Strickroth <email@cs-ware.de>
@csware csware changed the title GitHub actions GitHub actions for pull-requests Jul 23, 2024
@csware csware changed the title GitHub actions for pull-requests GitHub actions for pull requests Jul 23, 2024
@rbri
Copy link
Member

rbri commented Jul 26, 2024

Thanks a lot,

personally i'm not a big fan of all these github actions (because of the missing result visualizations) but i think this will definitely help.

Maybe we can introduce branch builds (like here https://jenkins.wetator.org/job/HtmlUnit%20-%20Library%20Tests/) for the core test (https://jenkins.wetator.org/job/HtmlUnit%20-%20Headless/) suite on jenkins also

@rbri rbri merged commit c024d39 into HtmlUnit:master Jul 26, 2024
7 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