Conversation
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1221Or
iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1221" |
|
What's the process with failing CI checks? Does someone restart them? The changes look good to me. The settings file is the old format, we should update the skill to generate the new one, but it's still valid. |
| "appHostPath": "../apphost.ts", | ||
| "language": "typescript/nodejs", | ||
| "packages": { | ||
| "CommunityToolkit.Aspire.Hosting.Java": "" |
There was a problem hiding this comment.
This doesn't work outside of the aspire repo. This needs to be a project reference.
There was a problem hiding this comment.
Bad skill that was made for our repos :/
|
I gotta ask, why Java first? Why not all of the other database integrations. |
@davidfowl - I did all of them: https://github.com/CommunityToolkit/Aspire/issues?q=is%3Apr+is%3Aopen+author%3AIEvangelist |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
eb5d108 to
6487997
Compare
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Minimum allowed line rate is |
Adds AspireExport coverage and a matching TypeScript validation apphost for java.