-
Notifications
You must be signed in to change notification settings - Fork 5
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
JN-372, DSEC-659 include sonar in Java CI #483
Merged
Merged
Changes from all commits
Commits
Show all changes
86 commits
Select commit
Hold shift + click to select a range
f1f6bef
include sonar in Java CI
TomConner ea49376
sonar in Java CI
TomConner d0fbe58
separate project sonar scans
TomConner 2781dab
sonar-project.properties at root
TomConner 7023028
combine all java and combine all typescript
TomConner 50add44
sonar typescript
TomConner c6963e4
sonar args in workflow
TomConner 64d6e2d
sonarscanner
TomConner 4810928
work in progress
TomConner dec810d
multiproject sonar
TomConner b6ebf5f
gradle sonar on single bound project
TomConner 1fe02e2
exclude spotless
TomConner becb898
include resources in sonar
TomConner 6cb22dc
include resources in sonar
TomConner 288039f
resources
TomConner b558e2c
gha sonar
TomConner bbdde71
gha
TomConner 150f883
typescript sonar specify branch
TomConner 5739af6
sonar on push
TomConner 67f6447
sonar on push
TomConner a1b489d
sonar on push
TomConner a1eee5d
sonar in Java CI
TomConner d3313bc
separate project sonar scans
TomConner 6019c52
sonar-project.properties at root
TomConner 69126f4
combine all java and combine all typescript
TomConner 6b6cff1
sonar typescript
TomConner cc8dbba
sonar args in workflow
TomConner c0aab52
sonarscanner
TomConner 3fd9559
work in progress
TomConner 24d44e6
multiproject sonar
TomConner f8e4d1f
gradle sonar on single bound project
TomConner 25e12fe
exclude spotless
TomConner 14c05f1
include resources in sonar
TomConner 166fba7
include resources in sonar
TomConner 11de4de
resources
TomConner bf92e87
gha sonar
TomConner 5389f6c
gha
TomConner ec0775c
typescript sonar specify branch
TomConner e240fc7
sonar on push
TomConner 5d8f8b7
sonar on push
TomConner 63530f5
sonar on push
TomConner 7a92258
announce message
TomConner d393b89
Merge branch 'development' into DSEC-659
TomConner c779f1c
merge
TomConner fe33a0c
revert
TomConner 7743265
separate project sonar scans
TomConner 7d57f7d
sonar-project.properties at root
TomConner 9a051f7
combine all java and combine all typescript
TomConner 77927be
sonar typescript
TomConner 9564e2f
sonar args in workflow
TomConner 6e9ce5c
sonarscanner
TomConner b26bc64
multiproject sonar
TomConner 8f95872
gradle sonar on single bound project
TomConner 7139df9
exclude spotless
TomConner 44739e2
include resources in sonar
TomConner 014f93a
include resources in sonar
TomConner 46043ed
resources
TomConner a573adb
gha sonar
TomConner cfe6b37
gha
TomConner 51f0e3f
typescript sonar specify branch
TomConner c49b612
sonar on push
TomConner 8d1492a
sonar on push
TomConner 61b1e54
sonar on push
TomConner 9d2a533
announce message
TomConner 9e76718
[JN-490] avoiding error on dupe mailing list signup (#482)
devonbush e167a32
[JN-477] admin help for export (#481)
devonbush 277da84
[JN-453] Preserve fields when switching question types in new questio…
MatthewBemis edf05e8
multiproject sonar
TomConner ae9c4a5
gradle sonar on single bound project
TomConner a5b8fe8
include resources in sonar
TomConner d6cfbc8
include resources in sonar
TomConner ff52e8c
resources
TomConner e379527
gha sonar
TomConner d2f7e14
gha
TomConner 3041c6b
typescript sonar specify branch
TomConner 088f9ec
sonar on push
TomConner e038f28
sonar on push
TomConner 1198dc1
sonar on push
TomConner 01bec4d
revert
TomConner ff4cd25
Merge branch 'DSEC-659' of github.com:broadinstitute/juniper into DSE…
TomConner bf7caa9
restore
TomConner 1a4f8f3
link to branch sonar scan
TomConner 5012efc
announce branch scan
TomConner 46e4d67
announce branch scan
TomConner 1305120
add ui-core
TomConner ae4a1c2
add ui-core
TomConner 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 |
---|---|---|
@@ -1,24 +1,67 @@ | ||
name: Sonar | ||
on: workflow_dispatch | ||
on: | ||
workflow_dispatch: | ||
push: | ||
branches: | ||
- master | ||
pull_request: | ||
types: [opened, synchronize, reopened] | ||
|
||
jobs: | ||
sonar: | ||
sonar-java: | ||
name: SonarCloud Java | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v3 | ||
|
||
- name: Set up JDK | ||
with: | ||
fetch-depth: 0 | ||
- name: Set up JDK 17 | ||
uses: actions/setup-java@v3 | ||
with: | ||
java-version: '17' | ||
distribution: 'temurin' | ||
cache: 'gradle' | ||
distribution: 'adopt' | ||
- name: Scan via gradle | ||
run: >- | ||
./gradlew | ||
--build-cache | ||
-x test | ||
-x spotlessJava | ||
-x spotlessJavaApply | ||
-x spotlessJavaCheck | ||
-x spotlessCheck | ||
-x spotlessApply | ||
build sonar -info | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any | ||
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} | ||
|
||
- name: Build all projects without running tests | ||
run: ./gradlew --build-cache compileJava | ||
|
||
- name: SonarCloud Scan | ||
sonar-typescript: | ||
name: SonarCloud TypeScript | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
subproject: ['ui-admin', 'ui-core', 'ui-participant'] | ||
steps: | ||
- uses: actions/checkout@v3 | ||
with: | ||
fetch-depth: 0 | ||
- name: Get branch name | ||
id: branch-name | ||
uses: tj-actions/branch-names@v5 | ||
- name: Scan ${{ matrix.subproject }} via sonarcloud-github-action | ||
uses: SonarSource/sonarcloud-github-action@master | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any | ||
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} | ||
with: | ||
projectBaseDir: ${{ github.workspace }}/${{ matrix.subproject }} | ||
args: > | ||
-Dsonar.branch.target=${{ github.event.repository.default_branch }} | ||
-Dsonar.branch.name=${{ steps.branch-name.outputs.current_branch }} | ||
- name: Announce | ||
run: | | ||
gh pr comment ${{ github.event.pull_request.number }} \ | ||
--body "SonarCloud [analyzed ${{ matrix.subproject }} on ${{ steps.branch-name.outputs.current_branch }}](https://sonarcloud.io/summary/new_code?id=broadinstitute_juniper-${{ matrix.subproject }}&branch=${{ steps.branch-name.outputs.current_branch }})" | ||
env: | ||
GH_TOKEN: ${{ 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 |
---|---|---|
|
@@ -42,3 +42,9 @@ npm-debug.log* | |
.scannerwork | ||
.sonar-token | ||
tmp/ | ||
|
||
# direnv | ||
.envrc | ||
|
||
*.hprof | ||
|
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
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 |
---|---|---|
|
@@ -55,4 +55,3 @@ dependencies { | |
test { | ||
useJUnitPlatform () | ||
} | ||
|
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 |
---|---|---|
|
@@ -35,4 +35,3 @@ dependencies { | |
test { | ||
useJUnitPlatform () | ||
} | ||
|
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 |
---|---|---|
@@ -1,4 +1,4 @@ | ||
rootProject.name = 'pearl' | ||
rootProject.name = 'juniper' | ||
include('core', 'populate', 'api-admin', 'api-participant') | ||
|
||
gradle.ext.releaseVersion = '0.0.85' |
This file was deleted.
Oops, something went wrong.
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,3 @@ | ||
sonar.projectKey=broadinstitute_juniper-ui-admin | ||
sonar.organization=dsp-appsec | ||
sonar.projectName=juniper-ui-admin |
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,3 @@ | ||
sonar.projectKey=broadinstitute_juniper-ui-core | ||
sonar.organization=dsp-appsec | ||
sonar.projectName=juniper-ui-core |
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,3 @@ | ||
sonar.projectKey=broadinstitute_juniper-ui-participant | ||
sonar.organization=dsp-appsec | ||
sonar.projectName=juniper-ui-participant |
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.
Should this be triggered by pushes to the
development
branch instead?development
is ultimately what ends up going to production, as we don't use amaster
ormain
branch.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. Good catch.