Skip to content

refactor(tools): move test_template.py into tools/ci, drop template_smoke.sh - #3

Merged
ZheFeng7110 merged 3 commits into
mcpplibs:mainfrom
ZheFeng7110:refactor/move-ci-scripts
Nov 6, 2026
Merged

ZheFeng7110 merged 3 commits into
mcpplibs:mainfrom
ZheFeng7110:refactor/move-ci-scripts

Conversation

@ZheFeng7110

@ZheFeng7110 ZheFeng7110 commented Nov 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Move tools/test_template.py into tools/ci/ next to verify_generated.py and update_mcpp_index.py; it is only run by the examples job in tests.yml. Repository root is now resolved via parents[2].
  • Remove tools/template_smoke.sh. Since the mwrap migration it only execed test_template.py, and nothing (CI, scripts, docs) called it.
  • tools/check_mwrap_release.py stays where it is: besides CI and release.yml, tools/package-source.sh runs it too.
  • Update references in tests.yml, the .gitignore comment and docs/architecture{,.zh}.md.

Test plan

  • python3 tools/ci/test_template.py --help
  • python3 tools/check_mwrap_release.py unchanged and passing ([email protected])
  • CI examples job (template smoke) green

Move test_template.py, check_mwrap_release.py and template_smoke.sh next to
the other CI entry points, adjust their repository-root resolution, and
update the workflows, package-source.sh, .gitignore and architecture docs.
…in tools/

template_smoke.sh only forwarded to test_template.py and had no callers.
check_mwrap_release.py is also run by tools/package-source.sh, so it is not
CI-only and stays at its original path.
@ZheFeng7110 ZheFeng7110 changed the title refactor(tools): move CI-only scripts into tools/ci refactor(tools): move test_template.py into tools/ci, drop template_smoke.sh Nov 6, 2026
@ZheFeng7110
ZheFeng7110 merged commit 34a5e57 into mcpplibs:main Nov 6, 2026
20 checks passed
@ZheFeng7110
ZheFeng7110 deleted the refactor/move-ci-scripts branch November 6, 2026 14:58
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