Skip to content

Comments

Fix CI/CD workflows: update build configuration, remove obsolete Code…#130

Merged
Stateford merged 3 commits intomasterfrom
fix-broken-cicd
Jan 19, 2026
Merged

Fix CI/CD workflows: update build configuration, remove obsolete Code…#130
Stateford merged 3 commits intomasterfrom
fix-broken-cicd

Conversation

@Stateford
Copy link
Owner

…QL and pylint workflows

Copilot AI review requested due to automatic review settings January 19, 2026 02:28
Copy link

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

This PR modernizes the CI/CD workflows by removing obsolete security and linting workflows and updating the build workflow with newer action versions and simplified configuration.

Changes:

  • Removed pylint.yml workflow (containing pylint, mypy, flake8, and bandit checks)
  • Removed codeql.yml workflow (containing CodeQL security scanning)
  • Updated build.yml with newer GitHub Actions versions and cleaner syntax

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/pylint.yml Complete removal of Python linting workflow
.github/workflows/codeql.yml Complete removal of CodeQL security scanning workflow
.github/workflows/build.yml Updated to use actions/checkout@v4, actions/setup-python@v5, actions/upload-artifact@v4, and microsoft/setup-msbuild@v2; simplified paths and conditionals; restricted tag pattern to 'v*' prefix

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

tags:
- '*'
branches: [master, develop]
tags: ['v*']
Copy link

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

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

The tag trigger pattern has been changed from '' (matching all tags) to 'v' (matching only tags starting with 'v'). This will cause the workflow to only trigger for tags like 'v1.0.0', 'v2.1.3', etc., and will not trigger for tags without the 'v' prefix. Ensure this is intentional and that your tagging strategy uses the 'v' prefix convention.

Suggested change
tags: ['v*']
tags: ['*']

Copilot uses AI. Check for mistakes.
@Stateford Stateford merged commit 607d7d8 into master Jan 19, 2026
2 checks passed
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.

1 participant