-
Notifications
You must be signed in to change notification settings - Fork 4
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
Artifact Publishing, Default Property Value Change, Unit Tests & Documentation Updates #64
Merged
Merged
Changes from 24 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
7dde93f
test artifact publishing
Michael7371 35b7f18
fix artifact endpoint
Michael7371 5460c51
update trigger to only run on release
Michael7371 3462850
testing -DremoveSnapshot task for artifact publishing github action
Michael7371 ebcc32d
updating artifact-publish github action to be triggered on release cr…
Michael7371 465545c
testing snapshot publishing
Michael7371 18ab77c
Removing GitHub token secret reference in workflow
Michael7371 50c31e3
Removing dev trigger as tags are immutable in GitHub's Maven Registry.
Michael7371 cfcef3c
updating workflow to only trigger on release tags
Michael7371 f0da6da
Changed default value for enable.auto.commit property to 'true'
dmccoystephenson da54ae1
Add unit tests
mwodahl d2bf634
comment update
mwodahl edc418e
Merge pull request #25 from CDOT-CV/config/default-auto-commit-to-true
payneBrandon 089318f
Merge pull request #23 from CDOT-CV/cicd-artifact-publishing
payneBrandon 2e06875
Merge branch 'dev' into Feature/Add-Unit-Tests
mwodahl 13e8d74
Update mockito verifications to work with Java Test Runner
mwodahl 1c24623
Add argLine, jacoco.version to pom.xml
mwodahl 44979a5
revert java.version to maven.compiler.source and maven.compiler.target
mwodahl 05cc92d
Merge pull request #26 from CDOT-CV/Feature/Add-Unit-Tests
drewjj df4901a
Revised README
dmccoystephenson 7c9d4a5
Merge pull request #27 from CDOT-CV/docs/review-docs-for-accuracy
payneBrandon afbd97a
Changed version to 1.6.0-SNAPSHOT
dmccoystephenson 3353541
Added release notes for version 1.6.0
dmccoystephenson 737e2fa
Merge pull request #28 from CDOT-CV/q3-release/version-change-and-rel…
drewjj 7abc88f
Removed unnecessary declaration from `getEnvironmentVariable()` metho…
dmccoystephenson ccc8ed4
Merge pull request #29 from CDOT-CV/pr/address-usdot-comments-9-19-2024
dmccoystephenson 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 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,31 @@ | ||
name: Publish Java Package | ||
|
||
on: | ||
push: | ||
tags: | ||
- 'jpo-s3-deposit-*' | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up JDK 21 | ||
uses: actions/setup-java@v4 | ||
with: | ||
java-version: '21' | ||
distribution: 'adopt' | ||
|
||
- name: Remove snapshot from version | ||
run: mvn versions:set -DremoveSnapshot | ||
|
||
- name: Build with Maven | ||
run: mvn -B package --file pom.xml | ||
|
||
- name: Publish to GitHub Packages | ||
run: mvn --batch-mode -Dgithub_organization=${{ github.repository_owner }} deploy | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
This file contains 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,15 @@ | ||
{ | ||
"java.test.config": { | ||
"env": { | ||
"TEST_VARIABLE": "testValue", | ||
"TEST_VARIABLE_EMPTY": "", | ||
"AWS_ACCESS_KEY_ID": "testAccessKey", | ||
"AWS_SECRET_ACCESS_KEY": "testSecretKey", | ||
"AWS_EXPIRATION": "2020-01-01 00:00:00", | ||
"AWS_SESSION_TOKEN": "testSessionToken", | ||
"API_ENDPOINT": "testApiEndpoint", | ||
"CONFLUENT_KEY": "testConfluentKey", | ||
"CONFLUENT_SECRET": "testConfluentSecret", | ||
} | ||
}, | ||
} |
This file contains 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 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 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 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
Oops, something went wrong.
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.
What will be the approx. artifact size and where we are using this ? As our ODE GitHub org is on free plan we have artifact storage limit of 500mb.
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.
The project's JAR file is about 14 KB in size. It is not currently being used, but it is for potential future use.
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.
Same comment from jpo-security-svcs seems like for Maven packages and other packages like npm, docker, etc storage limit will be not applied for public repo's. so I think we should be good to use artifacts. However for any private repo's this limits will apply. We have a backlog item for supporting GitHub artifacts setup in future release. Thanks!