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

Setup GH Workflow for PRs #2159

Merged
merged 2 commits into from
Mar 24, 2022
Merged

Setup GH Workflow for PRs #2159

merged 2 commits into from
Mar 24, 2022

Conversation

pzygielo
Copy link
Contributor

@pzygielo pzygielo commented Mar 23, 2022

I propose for consideration GitHub workflow to run for Pulls, as I can't find job with similar function on ci.eclipse.org.

There are two jobs defined:

  1. Perform build without running tests.
  2. If above completes successfully - build with tests this time.

I decided to split (temporarily) the build to two jobs, as I find many tests failing from time to time. The problem might be with the tests, might be with the grizzly itself.
Thus the failing job will be left to the discretion of reviewer to decide/ignore.

Defines
- maven.test.redirectTestOutputToFile=true
- forkCount=1
- reuseForks=false
as used in https://ci.eclipse.org/grizzly/job/Grizzly-master_nightly
@pzygielo pzygielo marked this pull request as ready for review March 23, 2022 09:55
@pzygielo
Copy link
Contributor Author

May I ask for review, please?

Copy link
Contributor

@kofemann kofemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

.github/workflows/maven.yml Show resolved Hide resolved
@arjantijms arjantijms merged commit 53e4350 into eclipse-ee4j:master Mar 24, 2022
@pzygielo pzygielo deleted the ghwf branch March 24, 2022 10:42
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.

3 participants