Skip to content

docs: correct built-in workflow step source paths - #5059

Merged
mnriem merged 1 commit into
github:mainfrom
WOLIKIMCHENG:docs/workflow-step-source-path
Oct 22, 2026
Merged

mnriem merged 1 commit into
github:mainfrom
WOLIKIMCHENG:docs/workflow-step-source-path

Conversation

@WOLIKIMCHENG

Copy link
Copy Markdown
Contributor

Description

Correct the two built-in source-directory references in design/workflow-step.md from steps/ to step/, matching src/specify_cli/workflows/step/. The installed-package path .specify/workflows/steps/<id>/ is unchanged.

Testing

  • Verified that only the two intended references changed and that the source path and relative links exist.
  • git diff --check

AI Disclosure

  • I did use AI assistance (described below)

AI disclosure: AI-assisted tools were used to correct the documentation and verify the changed paths.

Assisted-by: Codex CLI (model: gpt-6.1-sol, autonomous)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Both corrected references match the existing source packages and registration imports, with no executable changes or unresolved issues.

No open issues 🎉

What changed in this PR

Aligns the workflow-step design documentation with Specify CLI’s existing built-in package layout.

Changes:

  • Corrects both built-in source references from steps/ to step/.
  • Leaves the installed-package path .specify/workflows/steps/<id>/ unchanged.
File Description
design/​workflow-step.md Corrects source paths in the registration table and step-implementation instructions.

Review effort: Max

@mnriem mnriem added the triage-nice-to-have Verdict: evidence-backed fix or greenlit feature — land after review label Oct 22, 2026
@mnriem
mnriem merged commit 00c0aa2 into github:main Oct 22, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage-nice-to-have Verdict: evidence-backed fix or greenlit feature — land after review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants