Skip to content

fix(init): avoid duplicated okdev manifest paths#69

Merged
acmore merged 1 commit into
mainfrom
fix-folder-config-manifest-path
Apr 12, 2026
Merged

fix(init): avoid duplicated okdev manifest paths#69
acmore merged 1 commit into
mainfrom
fix-folder-config-manifest-path

Conversation

@acmore
Copy link
Copy Markdown
Owner

@acmore acmore commented Apr 12, 2026

Summary

  • resolve init scaffold output paths through one helper
  • keep .okdev/... manifest paths project-root relative when config is .okdev/okdev.yaml
  • cover both built-in PyTorchJob scaffolding and template-declared companion files

Tests

  • go test ./internal/cli -run 'TestInit.*(FolderConfigAndDotOkdevPath|ScaffoldsPyTorchJobManifest|CustomTemplateRendersDeclaredFile)'
  • go test ./internal/config ./internal/cli
  • .venv/bin/pre-commit run --all-files --hook-stage manual okdev-gofmt

@acmore acmore merged commit 242a045 into main Apr 12, 2026
1 check passed
@acmore acmore deleted the fix-folder-config-manifest-path branch April 12, 2026 09:23
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.

1 participant