-
Notifications
You must be signed in to change notification settings - Fork 42
Use runtime-import for markdown body instead of inlining (with recursive processing) #12687
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ort helper Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot do not write markdown body to a separate file, parse the body from the existing workflow file on disk. |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…ontent Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…eparate body files Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in commit 245da37. The compiler now uses |
…mplate wrapping support Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot merge main and recompile |
…e for workflows with imports Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot merge main and recompile |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot the imports should be inlined as it was before. The only file that changes treatment is the main agentic workflow |
…bitrary conditionals Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot make sure that runtime-import calls are processed recursively and only imported once |
|
@copilot merge main and recompile |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in commit 7a551a5. Runtime-import macros are now:
All 120 tests pass, including 11 new tests covering recursive imports, caching, and circular dependency detection. |
…ive processing) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…er-markdown-rendering # Conflicts: # .github/workflows/dependabot-burner.lock.yml # .github/workflows/test-project-url-default.lock.yml
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
Agent Container Tool Check ✅
Result: 10/12 tools available Missing Tools:
Core Development Tools: All essential tools (bash, git, jq, yq, curl, gh, node, python3, go) are available and working correctly.
|
Smoke Test Results: ✅ PASSPR Titles Reviewed:
Tests: Status: All tests passed @pelikhan
|
|
Fix test capturing stdout when version output moved to stderr
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Complete: Merged Main and Recompiled ✅
Merge Status
Merge Details
Conflicts Resolved:
.github/workflows/dependabot-burner.lock.yml.github/workflows/test-project-url-default.lock.ymlBoth files were regenerated during recompilation, so conflicts were resolved by accepting current version and recompiling.
Recompilation Results
Implementation Status
All features complete and working:
Ready for Review
The branch is now up-to-date with main and all workflows have been recompiled with the latest changes.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Changeset