prql icon indicating copy to clipboard operation
prql copied to clipboard

Add our premium PRQL Pro Tools VS Code IDE extension to your related tools and links in Explore section

Open RandomFractals opened this issue 2 years ago • 7 comments
trafficstars

What's up?

We published our Premium PRQL Pro Tools VS Code extension with PRQL Run Query Code Lens for the popular SQL Tools extension millions of developers and data scientists use back in March, after retrofitting and adding many additional features to your main PRQL VS Code extension earlier this year.

While we are looking for some individual and corporate sponsors to help us maintain and develop some of our new Pro Data Tools for VS Code IDE, we think it would be beneficial to the PRQL dev community and fans to have a link in your docs to our new PRQL Pro Tools that provide seamless integrate with SQL Tools extension many devs and data scientists use with over 20 different databases and drivers/plugins to explore various data sources in VS Code IDE.

Simple list item addition pointing interested users to the PRQL Pro Tools docs in your Explore README.md section in this repo should suffice:

Brief demo of PRQL Run Query Code Lens integrated with our DuckDB SQL Tools below. That Run PRQL Query code lens works in similar fashion with the other SQL Tools database drivers many of which you already support.

PRQL Code Lens

@max-sixty Would be happy to PR this addition to your README.md Explore section in this repo.

As a token of appreciation for listing PRQL Code Lens in your docs, we'd be open to contribute a PR for https://github.com/PRQL/prql-vscode/issues/165 and work out whatever other lingering disconnects we might have to wrap up supporting PRQL v0.9.5 in VS Code IDE.

RandomFractals avatar Oct 25 '23 14:10 RandomFractals

Could we start by adding it to the Readme for https://github.com/PRQL/prql-vscode? We could also add a sentence to https://prql-lang.org/book/project/integrations/vscode.html?

(I'm less keen on adding tools which aren't owned by the project to the Explore in the Readme, I hope the other options are reasonable for the moment. Happy to reassess in the future)

I also recognize that we've occasionally had some frictions between us — I do want to make clear that we might change our decision here (and I'm not the only voice). Can you confirm that you'll accept the decision gracefully if we do decide to not include it in the future?

max-sixty avatar Oct 25 '23 19:10 max-sixty

@max-sixty not sure what frictions you mention after we beefed up your official vs code prql ext. last Feb. Would be open to discuss that in some other private DM channels. We still feel very strongly what we proposed above should be listed in your core prql repo.

Besides, some of the links you have in your Explore section now are not really owned by your team per our read ...

image

RandomFractals avatar Oct 25 '23 20:10 RandomFractals

Besides, some of the links you have in your Explore section now are not really owned by your team per our read ...

Those are all owned by the project, no? Which is not?


Fine if no frictions! Just want to ensure you'd be OK if we changed how things were done in the future — would you be?

Assuming so — can we start with listing it in those places I suggested?

max-sixty avatar Oct 25 '23 21:10 max-sixty

ok. Let's add it to the PRQL Book VS Code ext. page and vscode ext. repo for now. We'll ref this ticket in those PRs.

PRQL VS Code extension doesn't have a good section like Explore in this repo. Why we thought it would be best to add it here. We can revisit that later. We just want to see if any of the PRQL VS Code extension users would be interested in direct SQL Tools and DB drivers integration.

How about we'll add new section there after https://github.com/PRQL/prql-vscode#prql-target. We can call it SQL Tools Integration or something along those lines, and add brief info about SQL Tools ext. and Run Query code lens, linked to the ext. doc we shared above.

RandomFractals avatar Oct 26 '23 12:10 RandomFractals

anyhoo, just an fyi: https://github.com/mtxr/vscode-sqltools/pull/1204

RandomFractals avatar Oct 26 '23 16:10 RandomFractals

anyhoo, just an fyi: mtxr/vscode-sqltools#1204

That would be ideal!

max-sixty avatar Oct 26 '23 17:10 max-sixty

That would be ideal!

It's been merged in SQL Tools main README.md already. Just scroll down to the Related Tools section in docs 😉

https://github.com/mtxr/vscode-sqltools

RandomFractals avatar Oct 26 '23 17:10 RandomFractals