Fix CI/CD workflows: update build configuration, remove obsolete Code…#130
Fix CI/CD workflows: update build configuration, remove obsolete Code…#130
Conversation
…QL and pylint workflows
There was a problem hiding this comment.
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*'] |
There was a problem hiding this comment.
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.
| tags: ['v*'] | |
| tags: ['*'] |
…witch to FetchContent for Google Test
…QL and pylint workflows