Skip to content

Add workflows for Codecov, CodeQL, and Codacy integration#3

Open
marekdkropiewnicki-dotcom wants to merge 9 commits intocodestorm-official:mainfrom
marekdkropiewnicki-dotcom:main
Open

Add workflows for Codecov, CodeQL, and Codacy integration#3
marekdkropiewnicki-dotcom wants to merge 9 commits intocodestorm-official:mainfrom
marekdkropiewnicki-dotcom:main

Conversation

@marekdkropiewnicki-dotcom
Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings April 22, 2026 21:03
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds GitHub Actions workflows and configuration to integrate code scanning and code quality/coverage tooling (CodeQL, Codecov, Codacy) plus Dependabot.

Changes:

  • Add CodeQL “Advanced” code scanning workflow.
  • Add pytest coverage workflow that uploads reports to Codecov.
  • Add Codacy security scan workflow and a Dependabot configuration file.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File Description
.github/workflows/codeql.yml Introduces CodeQL scanning job for Actions + Python (currently has YAML indentation issues).
.github/workflows/codecov.yml Adds a pytest coverage job and uploads to Codecov (may fail in repos without tests / fork PRs).
.github/workflows/codacy.yml Adds Codacy SARIF generation + upload to code scanning (may fail on fork PRs due to secrets).
.github/dependabot.yml Adds Dependabot config (currently invalid due to empty ecosystem).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/codecov.yml Outdated
Comment thread .github/dependabot.yml Outdated
Comment thread .github/workflows/codacy.yml
Comment thread .github/workflows/codeql.yml Outdated
Comment on lines +58 to +60
steps:
- name: Checkout repository
uses: actions/checkout@v4
Copy link

Copilot AI Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

steps: is defined, but the step items (- name: etc.) are not indented beneath it, which will cause a YAML parse error and prevent the workflow from running. Indent all - name: entries under steps:.

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/codecov.yml Outdated
Comment on lines +28 to +32
- name: Upload coverage reports to Codecov
uses: codecov/codecov-action@v5
with:
token: ${{ secrets.CODECOV_TOKEN }}
slug: marekdkropiewnicki-dotcom/flask-api
Copy link

Copilot AI Apr 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hardcoding slug: marekdkropiewnicki-dotcom/flask-api will break coverage uploads for forks and if the repository is renamed/transferred. The Codecov action can infer the repo from GITHUB_REPOSITORY, so it’s safer to remove the slug setting.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This workflow runs Bandit, a security linter for Python, on pushes and pull requests to the main branch, and on a scheduled basis.
Copy link
Copy Markdown
Author

@marekdkropiewnicki-dotcom marekdkropiewnicki-dotcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified

Copy link
Copy Markdown
Author

@marekdkropiewnicki-dotcom marekdkropiewnicki-dotcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified

Copy link
Copy Markdown
Author

@marekdkropiewnicki-dotcom marekdkropiewnicki-dotcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants