Conversation
Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
…8 for github-script) Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Simplify CI pipeline for testing and benchmarking
Fix and simplify CI pipeline for test and benchmark workflows
Feb 11, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the GitHub Actions CI workflow to better match the repository’s Go project layout and current testing/benchmark maturity, while simplifying execution and permissions.
Changes:
- Fixes Go toolchain version selection and updates test invocation to run across the whole module (
./...). - Simplifies CI by removing the OS matrix and running only on
ubuntu-latest. - Refactors benchmark reporting to a shorter inline comparison and posts results back to the PR.
Comments suppressed due to low confidence (1)
.github/workflows/ci.yml:106
- Posting a PR comment via
actions/github-scriptwill fail for pull requests coming from forks becauseGITHUB_TOKENcan’t be granted write permissions on thepull_requestevent in that scenario. Consider skipping this step whengithub.event.pull_request.head.repo.full_name != github.repository, or switching to apull_request_target-based approach (with appropriate hardening).
- name: Post benchmark results to PR
uses: actions/github-script@v8
with:
script: |
const fs = require('fs');
const comment = fs.readFileSync('benchmark-comment.md', 'utf8');
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Owner
|
@copilot Check the following files and try to adapt them to the repo |
…ndling Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
Contributor
Author
Done in 63b7d9b! I've adapted all 4 workflow files to the repository:
All workflows validated and ready to use. The files are adapted to the repository's structure (using |
Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
…th PR philosophy) Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
…convention Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
…ent branch Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
…ctly) Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
…-check Co-authored-by: adcondev <38170282+adcondev@users.noreply.github.com>
Copilot
AI
changed the title
Fix and simplify CI pipeline for test and benchmark workflows
Enhance CI/CD pipeline with comprehensive testing, security, and automation workflows
Feb 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR addresses critical configuration errors in the proposed CI pipeline and significantly expands CI/CD capabilities by integrating comprehensive testing, security scanning, and PR automation workflows.
Critical Fixes
'1.24'→'1.24.x'(correct semver syntax)./pkg/...→./...(project usesinternal/notpkg/)pull-requests: writefrom workflow-level to benchmark job only (least privilege)|| echo) that hid real failuresNew Workflows Added
This PR integrates 4 production-ready workflows (393 lines total):
1. ci.yml (163 lines) - Enhanced Test & Quality
2. codeql.yml (72 lines) - Security Scanning
3. pr-automation.yml (84 lines) - PR Management
4. pr-status-check.yml (74 lines) - Weekly Dashboard
Error Handling Philosophy
All workflows follow consistent error handling principles:
|| echo,|| true)Results
All workflows are adapted to repository structure (./..., Go 1.24.x, ubuntu-only), follow least privilege permissions, and are production-ready.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.