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

[ELY-2056] Add a new GitHub workflow to trigger CI for pull requests. #1475

Merged
merged 1 commit into from
Dec 15, 2020

Conversation

darranl
Copy link
Contributor

@darranl darranl commented Dec 15, 2020

https://issues.redhat.com/browse/ELY-2056

At the moment this is just a minimal PR to get things going, to test and tweak the PR triggers we really need it set up against a repo that receives PRs to get used to how it behaves so we know what changes it needs.

I haven't added additional items yet for gathering any logs etc.. we can see if we need those but for now all failures I have seen were reproducible locally using a download of the Zulu JDK.

Also for now I have included macOS in the list, we have a concurrency limit of 5 - we may take that one out later but for now contributors may develop on macOS so worth considering.

We already have Elytron Web split out into a separate project and we plan so split Jakarta EE and MicroProfile modules into their own projects so at some point I would like to explore how change in Elytron can be tested against those projects and how PRs against those projects can also build against the latest Elytron so interactions can be tested. Initially maybe that would be against PRs but if not maybe could be based on push triggers or be scheduled to run nightly.

@darranl
Copy link
Contributor Author

darranl commented Dec 15, 2020

FYI the "PR Request CI" builds will be expected to fail as we also need the changes in #1474

@fjuma fjuma added the +1 FJ label Dec 15, 2020
@fjuma fjuma merged commit fce7ad7 into wildfly-security:1.x Dec 15, 2020
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.

2 participants