(#5) Create placeholder workflows. - #6
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The new CI workflow name does not match the existing workflow_run trigger expectation ("ORCA CI"), which will prevent the dependency auto-merge workflow from running.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Creates initial GitHub Actions workflow files on main so workflows can run from feature branches, and removes legacy Acquia issue templates from the repository.
Changes:
- Adds a reusable-workflow wrapper to run the OCPL configuration management standards check on pull requests.
- Adds a placeholder CI workflow that runs on PRs, pushes to
main/release branches, and tags. - Removes the existing Acquia GitHub issue templates.
File summaries
| File | Description |
|---|---|
.github/workflows/ocpl_cm_standards_check.yml |
Introduces a PR-triggered workflow that delegates to a reusable OCPL standards-check workflow. |
.github/workflows/ci.yml |
Adds a placeholder CI workflow intended to exist on main for branch workflow execution. |
.github/ISSUE_TEMPLATE/support-request.md |
Removes legacy support request issue template. |
.github/ISSUE_TEMPLATE/feature_request.md |
Removes legacy feature request issue template. |
.github/ISSUE_TEMPLATE/bug_report.md |
Removes legacy bug report issue template. |
Review details
- Files reviewed: 5/5 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Also removes Acquia templates and workflows. Closes #5
69553c8 to
a5146b9
Compare
There was a problem hiding this comment.
🟡 Changes recommended
The PR makes additional workflow removals with operational impact that aren’t clearly reflected in the PR description/scope, and the reusable workflow pinning approach should be explicitly confirmed.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
- Files reviewed: 9/9 changed files
- Comments generated: 2
- Review effort level: Lite
| commitlint_remote: | ||
| uses: nciocpl/.github/.github/workflows/ocpl_cm_standards_check.yml@workflow/v4 | ||
| with: | ||
| config_ref: workflow/v4 |
| name: Main CI Workflow | ||
|
|
||
| on: |
Also removes Acquia templates and workflows.
Closes #5