-
-
Notifications
You must be signed in to change notification settings - Fork 1
x #96
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
x #96
Changes from all commits
789f799
06fd47f
f1cec12
14c6ae0
f002af9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| name: Qodana | ||
| on: | ||
| workflow_dispatch: | ||
| pull_request: | ||
| push: | ||
| branches: # Specify your branches here | ||
| - main # The 'main' branch | ||
| - 'releases/*' # The release branches | ||
|
|
||
| jobs: | ||
| qodana: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
| checks: write | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| with: | ||
|
DarkModder33 marked this conversation as resolved.
|
||
| ref: ${{ github.event.pull_request.head.sha }} # to check out the actual pull request commit, not the merge commit | ||
| fetch-depth: 0 # a full history is required for pull request analysis | ||
|
Comment on lines
+18
to
+21
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. issue: Using This expression is only defined for |
||
| - name: 'Qodana Scan' | ||
| uses: JetBrains/qodana-action@v2025.3 | ||
| with: | ||
| pr-mode: false | ||
| env: | ||
| QODANA_TOKEN: ${{ secrets.QODANA_TOKEN_1287543422 }} | ||
| QODANA_ENDPOINT: 'https://qodana.cloud' | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| #################################################################################################################### | ||
| # WARNING: Do not store sensitive information in this file, as its contents will be included in the Qodana report. # | ||
| #################################################################################################################### | ||
|
|
||
| version: "1.0" | ||
| linter: jetbrains/qodana-jvm:2025.3 | ||
| profile: | ||
| name: qodana.recommended | ||
| include: | ||
| - name: CheckDependencyLicenses |
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.
🚨 suggestion (security): Workflow permissions are broader than necessary for a read-only analysis job.
These permissions let the job modify repo contents and PRs. If Qodana doesn’t need to push commits or edit PR descriptions, consider reducing to
contents: readplus only the minimalpull-requests/checkspermissions required by its integration. This limits impact if the workflow or an action is compromised.