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

Submitted a new extension to the gallery #14926

Merged
merged 5 commits into from
Apr 8, 2021
Merged

Submitted a new extension to the gallery #14926

merged 5 commits into from
Apr 8, 2021

Conversation

ernstc
Copy link

@ernstc ernstc commented Mar 31, 2021

The extension submitted is "SQL Data Inspector" (sql-data-inspector).

This PR fixes #

The extension submitted is "SQL Data Inspector" (sql-data-inspector).
@ghost
Copy link

ghost commented Mar 31, 2021

CLA assistant check
All CLA requirements met.

Copy link
Contributor

@Charles-Gagnon Charles-Gagnon left a comment

Choose a reason for hiding this comment

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

Could you please :

  1. Add this to the insiders gallery as well
  2. Add a Microsoft.AzDataEngine property next to the Microsoft.VisualStudio.Code.Engine property (it can still be * since it doesn't look like you have any dependency on a specific version of ADS, but you should still have one regardless)

Added Microsoft.AzDataEngine property to the extension sql-data-inspector.
Copy link
Contributor

@Charles-Gagnon Charles-Gagnon left a comment

Choose a reason for hiding this comment

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

Played around with it a bit and didn't see any major issues, @alanrenmsft this look good to you?

@ernstc ernstc requested a review from abist as a code owner April 8, 2021 17:07
@Charles-Gagnon Charles-Gagnon merged commit b97602f into microsoft:release/extensions Apr 8, 2021
@Charles-Gagnon
Copy link
Contributor

@ernstc The gallery has been updated, thank you for your contribution!

@ernstc ernstc deleted the patch-1 branch April 8, 2021 23:17
abist pushed a commit that referenced this pull request Apr 13, 2021
* Update Arc & Azdata extensions - Insiders (#14990)

* Validate galleries for RC1 branch as well (#14994)

* Update to extensions gallery json with upcoming language pack update (#14924)

* Revert "Update to extensions gallery json with upcoming language pack update (#14924)" (#15032)

This reverts commit bea6526.

* Update azdata to latest - insiders (#15034)

* Update CODEOWNERS (#15038)

DRI

* Update Kusto to 0.5.2 (#14959)

* Updated Kusto version to 0.5.2.

* Updated minimum AzDataEngine version

* Update version of schema visualization (#15051)

* Submitted a new extension to the gallery (#14926)

* Submitted a new extension to the gallery

The extension submitted is "SQL Data Inspector" (sql-data-inspector).

* Fixed TotalCount in the extensions gallery.

* Added extension sql-data-inspector to the insider gallery.
Added Microsoft.AzDataEngine property to the extension sql-data-inspector.

* Updated version for extension "sql-data-inspector" to v0.1.1.

* Changed extension description and updated version to "0.1.2"

Co-authored-by: Alex Ma <alma1@microsoft.com>
Co-authored-by: Aditya Bist <adbist@microsoft.com>
Co-authored-by: Justin M <63619224+JustinMDotNet@users.noreply.github.com>
Co-authored-by: Jens Hunt <himself@jenshunt.se>
Co-authored-by: Ernesto Cianciotta <ernesto.cianciotta@gmail.com>
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.

4 participants