Skip to content

refactor: move templates under examples#32

Merged
timzsu merged 2 commits into
mainfrom
zsu/move-template-under-example
May 11, 2026
Merged

refactor: move templates under examples#32
timzsu merged 2 commits into
mainfrom
zsu/move-template-under-example

Conversation

@timzsu
Copy link
Copy Markdown
Collaborator

@timzsu timzsu commented May 11, 2026

Purpose

This PR moves all templates under examples, as called out in #22, and updates the references accordingly

Changes

  • templates/examples/templates/.
  • Repoint references:
    • Docs: README.md, AGENTS.md, docs/{ARCHITECTURE,CLI,SDK,WORKFLOWS}.md,
      src/worker/executors/agent/README.md.
    • Build / CI: .dockerignore exclude rule.
    • Tests: tests/server/task/test_template_validation.py TEMPLATE_DIR
      now points at examples/templates.
  • Directory map in docs/ARCHITECTURE.md collapses the old
    templates/ and examples/ rows into one examples/ entry.

Test Plan

  • uv run pytest tests/server tests/shared tests/sdk tests/cli — the
    template-parse parametrization picks up every YAML at the new path.

Test Result

474 passed (server + shared + sdk + cli).


Pre-submission Checklist
  • I have read the contribution guidelines.
  • I have run pre-commit run --all-files and fixed any issues.
  • I have added or updated tests covering my changes (if applicable).
  • I have verified that uv run pytest tests/ passes locally.
  • If I changed shared schemas or proto definitions, I have checked downstream compatibility across Server and Worker.
  • If I changed the SDK or CLI, I have verified the affected packages work (uv sync --all-packages --group ci --frozen).
  • If this is a breaking change, I have prefixed the PR title with [BREAKING] and described migration steps above.
  • I have updated documentation or config examples if user-facing behavior changed.

timzsu added 2 commits May 11, 2026 09:00
Signed-off-by: Zhengyuan Su <su.zhengyuan@u.nus.edu>
Repoint docs (README, AGENTS, CLI, SDK, ARCHITECTURE, WORKFLOWS, agent
executor README), the .dockerignore exclude rule, and the template
parser test's TEMPLATE_DIR at the new examples/templates/ location.

Signed-off-by: Zhengyuan Su <su.zhengyuan@u.nus.edu>
@timzsu timzsu marked this pull request as ready for review May 11, 2026 09:12
@timzsu timzsu requested a review from kaiitunnz May 11, 2026 09:12
@timzsu timzsu mentioned this pull request May 11, 2026
1 task
@kaiitunnz kaiitunnz changed the title refactor: Move templates under examples. refactor: move templates under examples. May 11, 2026
@kaiitunnz kaiitunnz changed the title refactor: move templates under examples. refactor: move templates under examples May 11, 2026
Copy link
Copy Markdown
Collaborator

@kaiitunnz kaiitunnz left a comment

Choose a reason for hiding this comment

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

LGTM.

@timzsu timzsu merged commit 2e40c6d into main May 11, 2026
12 of 13 checks passed
@timzsu timzsu deleted the zsu/move-template-under-example branch May 11, 2026 12:03
@timzsu timzsu linked an issue May 12, 2026 that may be closed by this pull request
1 task
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.

[RFC]: Cleanup workflow templates.

2 participants