Skip to content

docs: correct factual mismatches against the real CLI output - #22

Merged
Bangkah merged 1 commit into
mainfrom
docs/factual-fixes
Oct 22, 2026
Merged

Bangkah merged 1 commit into
mainfrom
docs/factual-fixes

Conversation

@Bangkah

@Bangkah Bangkah commented Oct 22, 2026

Copy link
Copy Markdown
Member

Doc-level facts were re-audited against the actual binary and source:

  • ci.yml, not ci.yaml (both Home table and Getting Started tree); generated tree uses lowercase kubernetes/ paths
  • Dockerfile non-root claim now says USER 1000:1000 (scratch base) instead of implying k8s securityContext; k8s manifest description now matches reality (resource requests/limits, probes only with --health-path, optional Ingress)
  • Validator page rewrote to the real rule set (Dockerfile FROM/login/shell-chaining checks; workflow single-doc name/on/jobs/checkout/SHA-tag; Deployment selector-consistency, unique container names, integer containerPort; Service port/targetPort; Ingress pathType + backend service wiring; cross-resource check) and exit rows (0/4/1/2 as implemented)
  • Wizard page steps corrected to the real dialog order (Stack, App type, Commands, Port, Registry, Image, Kubernetes, Summary)
  • Exit code 4 row broadened (render failure, missing artifacts, lint failures) and ci.yaml example removed
  • init callout: backup files are .devopsgen.bak with best-effort rollback
  • detect page: explicit exit code 5 for ambiguity/none

dist rebuilt and included.

Copilot AI balanced review requested due to automatic review settings October 22, 2026 14:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Bangkah
Bangkah merged commit 0660528 into main Oct 22, 2026
18 of 25 checks passed
@Bangkah
Bangkah deleted the docs/factual-fixes branch October 22, 2026 14:20
Sign up for free to 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.

2 participants