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

Add deploy options support #11567

Merged
merged 7 commits into from
Jul 30, 2020
Merged

Add deploy options support #11567

merged 7 commits into from
Jul 30, 2020

Conversation

kisantia
Copy link
Contributor

Adding support to pass deployment options for deploy and generate script operations

@coveralls
Copy link

coveralls commented Jul 29, 2020

Coverage Status

Coverage increased (+0.004%) to 37.621% when pulling 77de789 on kisantia/deployOptions into d244a14 on main.

@@ -21,6 +21,91 @@ export const mockDacFxResult = {
report: ''
};

export const mockDacFxOptionsResult: mssql.DacFxOptionsResult = {
Copy link
Contributor

Choose a reason for hiding this comment

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

I vaguely remember... did Charles had some way for common test file between extensions? that way you wont have to define this twice.

Copy link
Contributor

Choose a reason for hiding this comment

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

You can't share files directly currently.

Although at some point I did want to investigate having the extension tsconfigs pull in common files and compile them. I don't know if there's anything that would prevent it but theoretically that should work?

Otherwise your other options are :

  1. NPM package
  2. Expose stuff through extension API

Between those two #2 is probably the one I'd suggest if you felt this was really worth it. But in general for test stuff I think it's fine to just duplicate the code for now - especially if it's only going to be used by 2 extensions.

We can have further discussion on this though since it's a common thing we run in to and would be nice to have a solid answer for.

@kisantia kisantia merged commit c06bd08 into main Jul 30, 2020
@kisantia kisantia deleted the kisantia/deployOptions branch July 30, 2020 00:52
anthonydresser pushed a commit that referenced this pull request Jul 30, 2020
* add getOptionsFromProfile request

* update deploy and generate script to accept options

* fix tests

* update message

* update message to say what isn't supported

* bump sqltoolsservice version
aaomidi pushed a commit that referenced this pull request Jul 30, 2020
* add getOptionsFromProfile request

* update deploy and generate script to accept options

* fix tests

* update message

* update message to say what isn't supported

* bump sqltoolsservice version
aaomidi pushed a commit that referenced this pull request Jul 30, 2020
* add getOptionsFromProfile request

* update deploy and generate script to accept options

* fix tests

* update message

* update message to say what isn't supported

* bump sqltoolsservice version
halerankin pushed a commit that referenced this pull request Aug 3, 2020
* add getOptionsFromProfile request

* update deploy and generate script to accept options

* fix tests

* update message

* update message to say what isn't supported

* bump sqltoolsservice version
gupta1 pushed a commit that referenced this pull request Aug 3, 2020
* add getOptionsFromProfile request

* update deploy and generate script to accept options

* fix tests

* update message

* update message to say what isn't supported

* bump sqltoolsservice version
ktech99 pushed a commit that referenced this pull request Aug 23, 2020
* add getOptionsFromProfile request

* update deploy and generate script to accept options

* fix tests

* update message

* update message to say what isn't supported

* bump sqltoolsservice version
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