-
Notifications
You must be signed in to change notification settings - Fork 154
Idea to add CI pipeline with github actions #320
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
Closed
Closed
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
82746fa
Create build.yml
paulrutter dcc0980
Create root pom.xml
paulrutter 7260696
Update build.yml
paulrutter fba4fc2
Update build.yml
paulrutter 6a51efe
Update pom.xml
paulrutter abc7ecf
Update pom.xml
paulrutter 9ff29c9
Update build.yml
paulrutter 2a1e2b4
Update pom.xml
paulrutter 84000cc
Update build.yml
paulrutter cae3d08
Update build.yml
paulrutter 31cbcab
Update build.yml
paulrutter 2174c83
Update pom.xml
paulrutter a2a3ff3
Update build.yml
paulrutter 04d85b5
Update build.yml
paulrutter 1b1fb88
Update build.yml
paulrutter 1da24c0
Update build.yml
paulrutter 4d1c192
Update build.yml
paulrutter cd7df20
Update build.yml
paulrutter d74716d
Update build.yml
paulrutter 8044e9c
Set plugin version
paulrutter 0ab6886
Follow https://sling.apache.org/documentation/development/testing-pax…
paulrutter d785152
Update build.yml
paulrutter 8a43eaf
Remove combine
paulrutter 77c68d5
Update maven
paulrutter d2e958c
Remove root pom and replace with -f command
paulrutter 1651b59
Move configuration to execution
paulrutter bb6324b
Remove debug flag
paulrutter b1d3377
Use argLine (https://maven.apache.org/surefire/maven-failsafe-plugin/…
paulrutter 49c7359
Try env variables instead
paulrutter e624f79
Fix light test
paulrutter 46fb9d7
Add framework to test
paulrutter 2c7e21a
useSystemClassLoader
paulrutter 605848b
Update build.yml
paulrutter 5211fff
Update build.yml
paulrutter e54ac5f
Update build.yml
paulrutter cc33d82
Update build.yml
paulrutter 0a93c63
Update build.yml
paulrutter 9a85720
Update build.yml
paulrutter fa76f86
Update build.yml
paulrutter d961787
Update build.yml
paulrutter bcff47a
Update build.yml
paulrutter dc9f076
Update build.yml
paulrutter 82ec8d2
Update build.yml
paulrutter b6ae1a3
Update build.yml
paulrutter c196114
Update build.yml
paulrutter abf6a36
Update build.yml
paulrutter e450eab
Update build.yml
paulrutter 5bd087c
Update build.yml
paulrutter 92206e4
Update build.yml
paulrutter 8038ad9
Update build.yml
paulrutter ce1dbdb
Update pom.xml
paulrutter 5a60742
Update pom.xml
paulrutter 90aa249
Update build.yml
paulrutter 28343eb
Update build.yml
paulrutter f55faa1
Update build.yml
paulrutter c8ae47e
Update pom.xml
paulrutter 3a39471
Update build.yml
paulrutter e4035f1
Revert unnecessary changes
paulrutter 37cbfca
Revert unnecessary changes
paulrutter File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| # This workflow will build a Java project with Maven, and cache/restore any dependencies to improve the workflow execution time | ||
| # For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-java-with-maven | ||
|
|
||
| # This workflow uses actions that are not certified by GitHub. | ||
| # They are provided by a third-party and are governed by | ||
| # separate terms of service, privacy policy, and support | ||
| # documentation. | ||
|
|
||
| name: Java CI with Maven | ||
|
|
||
| on: | ||
| push: | ||
| branches: [ "master" ] | ||
| pull_request: | ||
| branches: [ "master" ] | ||
|
|
||
| jobs: | ||
| build: | ||
|
|
||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - name: Set up JDK 17 | ||
| uses: actions/setup-java@v3 | ||
| with: | ||
| java-version: '17' | ||
| distribution: 'temurin' | ||
| cache: maven | ||
| - name: Felix HTTP | ||
| run: mvn -B -DskipITs --file http clean install | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still required now
-DskipITsis used?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the regular unit tests also don't run with what's in master right now.