Skip to content

Fix/ci codacy github action failing due to multiple sarif #686

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.

Sign up for GitHub

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

Merged
juarezr merged 3 commits into from
Aug 13, 2025
Merged

Fix/ci codacy github action failing due to multiple sarif #686

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1a43a90
chore: trim vscode extension recommedation
juarezr Aug 13, 2025
c850fcc
fix: codacy ci workflow fails due to multiple sarit files
juarezr Aug 13, 2025
ab1d835
chore: add basic vscode settings for the editor
juarezr Aug 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .github/workflows/codacy-analysis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,18 @@ jobs:
# This will handover control about PR rejection to the GitHub side
max-allowed-issues: 2147483647

- name: Fetch SARIF files
uses: actions/download-artifact@v4
with:
path: ../results
pattern: *.sarif
merge-multiple: true

# Upload the SARIF file generated in the previous step
- name: Upload SARIF results file
uses: github/codeql-action/upload-sarif@v3
with:
sarif_file: results.sarif
category: codacy

## End of codacy-security-scan job ##
12 changes: 4 additions & 8 deletions .vscode/extensions.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
//
//-- Used for IDE, Workbench, Tools --------------------------------------------
//
"editorconfig.editorconfig",
//
//-- Used for linters, formatters ----------------------------------------------
//
Expand All @@ -12,18 +11,15 @@
"ms-python.vscode-python-envs",
"ms-python.debugpy",
"charliermarsh.ruff",
// "njpwerner.autodocstring",
// "njqdev.vscode-python-typehint",
//
// "ms-python.pylint",
// "ms-python.flake8",
// "ms-python.mypy-type-checker",
// "ms-python.isort",
//-- Used for: launch.json, tasks.json, launch/debug -----------------------
//
"rioj7.command-variable", //? Add files/pickers/commands variables for launch.json and tasks.json
"augustocdias.tasks-shell-input", //? Use shell commands as input for your tasks
//
//-- Used for: Git, Code Quality -----------------------------------------------
//
"Wequick.coverage-gutters",
// "vivaxy.vscode-conventional-commits"
],
"unwantedRecommendations": [
//
Expand Down
15 changes: 14 additions & 1 deletion .vscode/settings.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,24 @@
"editor.formatOnType": true,
"editor.tabSize": 4,
"editor.wordWrapColumn": 88,
"editor.renderWhitespace": "boundary",
"files.eol": "\n",
"files.encoding": "utf8",
"notebook.formatOnSave.enabled": false,
//
//-- Editor visual settings --------------------------------------------------------
//
"editor.fontLigatures": "'liga', 'calt', 'dlig', 'THND'",
"editor.renderWhitespace": "boundary",
"editor.guides.bracketPairs": true,
"editor.folding": true,
"editor.foldingStrategy": "auto",
"editor.showFoldingControls": "always",
"editor.rulers": [
88,
100,
120
],
//
//-- Editor settings for search/view files -----------------------------------------
//
"files.exclude": {
Expand Down
Loading