Skip to content
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

integrate data preview extension #242

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

ajshedivy
Copy link
Collaborator

Addresses #240

TODO:

  • Add command to Run Dialog to open Data Preview
  • review preview logic
  • add data preview extension as dependency, or check if the extension exists
    • show message to user that preview requires data preview extension installed
  • use vscode.fs instead of fs/promises

Copy link

👋 A new build is available for this PR based on 989b397.

@RandomFractals
Copy link

RandomFractals commented Jun 27, 2024

@ajshedivy great idea! Fair note: our data preview extension is very stale and out of date.

Let us know how this PR pans out for you. We've been meaning to publish new Data Preview Pro Tools for some time now, but currently taking a break from unpaid VS Code extensions dev till MSFT makes it possible for devs to monetize them directly from VS Code marketplace.

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