Skip to content

chore: upgrade to Vega-Lite v6 and Vega-Embed v7#81

Merged
thomasp85 merged 1 commit intoposit-dev:mainfrom
cpsievert:feature/vegalite-v6-upgrade
Jan 22, 2026
Merged

chore: upgrade to Vega-Lite v6 and Vega-Embed v7#81
thomasp85 merged 1 commit intoposit-dev:mainfrom
cpsievert:feature/vegalite-v6-upgrade

Conversation

@cpsievert
Copy link
Collaborator

@cpsievert cpsievert commented Jan 21, 2026

This upgrade will be useful for #74, where the current version of altair is expecting v6

  • Upgrade Vega-Lite from v5 to v6
  • Upgrade Vega-Embed from v6 to v7
  • Vega remains at v5 (no change required per Vega-Lite v6 release notes)

Changes

  • src/writer/vegalite.rs: Update schema URL and doc comment
  • ggsql-jupyter/src/display.rs: Update CDN URLs for vega-lite and vega-embed
  • ggsql-jupyter/tests/test_compliance.py: Update expected MIME type
  • ggsql-jupyter/tests/test_integration.py: Update expected MIME type
  • CLAUDE.md: Update documentation references

Test plan

  • All Rust tests pass (cargo test -p ggsql -p ggsql-jupyter)
  • Generated JSON has correct v6 schema URL
  • Manual verification in Vega-Lite editor confirms specs render correctly

🤖 Generated with Claude Code

Update visualization dependencies to latest major versions:
- Vega-Lite: v5 → v6
- Vega-Embed: v6 → v7
- Vega remains at v5 (no change required)

Changes:
- Update schema URL in VegaLiteWriter
- Update CDN URLs in Jupyter kernel display module
- Update MIME type expectations in Python tests
- Update documentation references

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@cpsievert cpsievert changed the title feat: upgrade to Vega-Lite v6 and Vega-Embed v7 chore: upgrade to Vega-Lite v6 and Vega-Embed v7 Jan 21, 2026
Copy link
Collaborator

@thomasp85 thomasp85 left a comment

Choose a reason for hiding this comment

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

LGTM

@thomasp85 thomasp85 merged commit 2ca16ce into posit-dev:main Jan 22, 2026
3 checks passed
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.

2 participants