Skip to content

Actions setup: remove CodeQL advanced setup#35

Merged
migue merged 1 commit into
mainfrom
git-systems/remove-codeql-advanced-action
Oct 31, 2025
Merged

Actions setup: remove CodeQL advanced setup#35
migue merged 1 commit into
mainfrom
git-systems/remove-codeql-advanced-action

Conversation

@migue

@migue migue commented Oct 31, 2025

Copy link
Copy Markdown
Contributor

The repository has configured a "Default" CodeQL setup but at the same time we provide a yaml file. It looks like it kind of confuses the backend.

Let's use the default setup for now

@migue migue requested a review from a team as a code owner October 31, 2025 08:44
Copilot AI review requested due to automatic review settings October 31, 2025 08:44
@migue migue self-assigned this Oct 31, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the CodeQL security analysis workflow from the repository. This eliminates automated code scanning for security vulnerabilities and code quality issues in the Go codebase.

  • Deletion of the entire CodeQL workflow configuration file
Comments suppressed due to low confidence (1)

.github/workflows/codeql.yaml:1

  • Removing the CodeQL workflow eliminates automated security scanning for the repository. This removes protection against common vulnerabilities and code quality issues. Consider keeping this workflow or providing an alternative security scanning solution to maintain code security standards.

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

@migue migue merged commit bdee96f into main Oct 31, 2025
11 checks passed
@migue migue deleted the git-systems/remove-codeql-advanced-action branch October 31, 2025 10:07
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.

4 participants