Skip to content

ci: PR template validation and workflow re-runs - #2215

Closed
ashleyshaw wants to merge 5 commits into
developfrom
feat/adr-phase-2-cli-commands-clean
Closed

ci: PR template validation and workflow re-runs#2215
ashleyshaw wants to merge 5 commits into
developfrom
feat/adr-phase-2-cli-commands-clean

Conversation

@ashleyshaw

Copy link
Copy Markdown
Member

Summary

Rebased feat/adr-phase-2-cli-commands-clean branch on latest develop.

Current branch status:

  • ✅ Successfully rebased on develop
  • 5 commits (CI workflow trigger commits)
  • 0 files with unique changes relative to develop

Commits

This branch contains CI-related commits focused on PR template validation:

  • ci: Add Test Plan section to PR template
  • ci: Trigger workflow re-run for PR template validation (multiple)
  • fix: Correct PR template section detection

Note

The ADR Phase 2 CLI implementation work may already be merged into develop. These commits appear to be maintenance/validation work only.

Linked Issues

Related to ADR Agent Phase 2 work and PR template standardization.

Checklist (Global DoD / PR)

  • Branch rebased on develop
  • Conflicts resolved
  • Ready for review
  • CI validation in progress

🤖 Generated with Claude Code

@github-actions

Copy link
Copy Markdown
Contributor

🚫 This PR description is missing required template content.

Missing required section(s): Linked issues, Changelog

Please update the PR body using one of the repository PR templates:

Empty placeholders, unchecked checklist boxes, and stub issue references do not count.

@github-actions

Copy link
Copy Markdown
Contributor

❌ Branch Name Validation Failed

The branch name feat/adr-phase-2-cli-commands-clean does not follow the LightSpeed branching strategy.

Required Format

{type}/{scope}-{short-title}
  • type: one of the allowed prefixes (lowercase)
  • scope: lowercase, hyphens only (no underscores or uppercase)
  • title: lowercase, hyphens only (no underscores or uppercase)

Allowed Branch Types

feat, fix, hotfix, release, refactor, chore, docs, test, perf, ci, build, deps, security, revert, research, design, a11y, ux, i18n, ops, proto, ds, api, schema, telemetry, content, seo, config, migrate, qa, uat, audit, codex

Valid Examples

  • feat/branch-naming-enforcement
  • fix/validation-script-bug
  • chore/update-dependencies
  • docs/branching-strategy-guide
  • hotfix/critical-security-patch

Invalid Examples

  • claude/my-branch (type "claude" not allowed)
  • Feature/MyBranch (uppercase not allowed)
  • fix-bug (missing type prefix)
  • feat/my_feature (underscores not allowed)
  • feat/MyFeature (uppercase not allowed)

Solution

Rename your branch to follow the pattern and update the PR.

For more information, see docs/BRANCHING_STRATEGY.md.

@ashleyshaw

Copy link
Copy Markdown
MemberAuthor

Duplicate of PR #2112, which has already been merged to develop. This PR uses the same branch (feat/adr-phase-2-cli-commands-clean) and contains the same ADR Agent Phase 2 CLI commands implementation.

Sign up for freeto 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

@ashleyshaw