Repository navigation
chore(skills): move data-explorer to Drop - #414
Conversation
It writes an HTML page from report.json rather than Comark, and its page ends up in Drop. Drop serves it now (vite-hub/drop ff069969), so comark-ui drops the skill, its docs page and its test, and the skills index and report point to Drop's.
|
The workflow's GitHub's error |
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
comark-annotate
comark-badge
comark-cards
comark-carousel
comark-cast
comark-charts
comark-chat
comark-chips
comark-cjk
comark-code
comark-code-notes
comark-colors
comark-comments
comark-critic
comark-deflist
comark-details
comark-diagrams
comark-diff
comark-embed
comark-fields
comark-figure
comark-file-tree
comark-footnotes
comark-github
comark-icon
comark-include
comark-keys
comark-links
comark-linter
comark-magic-move
comark-mark
markdown-it-to-comark
marked-to-comark
comark-mathml
comark-media
comark-metrics
migrate-to-comark
comark-outline
comark-properties
comark-qr
comark-quiz
comark-reading-time
remark-to-comark
comark-sources
comark-steps
comark-table
comark-tabs
comark-tests
comark-time
comark-ui
comark-xref
comark-zoom
commit: |
Latest result Pull request merged. Final answersSession Pull request merged. |

data-explorer(#410) writes an HTML page from areport.json, not Comark. It doesn't usecomark-chartseither: its template draws with Vega-Lite and only used comark-ui on jsDelivr to loadapp.jsandstyle.css. Its page ends up in Drop anyway, where a rebuild on new data becomes the next version at the same link.The skill now lives in Drop (vite-hub/drop@ff069969), which serves it at
/.well-known/skills/data-explorer/and as the fallback for a page's template. This PR removes the skill, its docs page and its test from comark-ui, and the skills index andreportpoint to Drop's. It has to merge before the v0.11.1 release (#408), so npm never ships the jsDelivr path that unfilled pages would depend on.The release notes in #408 still list #410, since uppt builds them from commits. That line should go when the release is cut.
Checked: the comark-ui build,
packages/ui/test/skills.test.tsandtest/workspace.test.ts(417 passed),pnpm run lint:docsandvp fmt --check. I didn't run the fullpnpm ready.