Update from upstream repo Microsoft/sqlopsstudio@master #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Using azure sql compatible query * Included changes requested in PR for azure specific DBs and then fixed issue #9361 for contained databases. * Made some changes mentioned in the PR - Converted let to const - Logging the error
* Update coveralls task * Undo sql change
* add tests for statusmodeselect * add another test
* extend widget container * remove title * Revert "extend widget container" * showTitle option * rename to showHeader * comments and refactoring
* Add a pop up for dotnet install if not present * Second option of default location for mac/linux * correcting the pop up message * remove extra dependency * updating as per PR comments
* fix errors in message panel * fix compile
Converted `test-extensions-unit.bat` to javascript to add a small QoL improvement that I think people will find useful: specifying which extension(s) you want to run tests for instead of commenting out lines of a script. In addition to existing behavior being mostly preserved (node test-extensions-unit.js defaults to running all unit tests), you can specify one or more: ``` node test-extensions-unit.js sql-database-projects node test-extensions-unit.js sql-database-projects schema-compare ```
…low (#10246) * fix overflow focus not always going to first element in overflow * adding tests
… tree. (#10214) * Added Edit Connection Command * Wip changes for new connection dialog * Testing * WIP commit * added ID check to ensure connection * wip commit * model id check implemented * addfooterbutton now accepts events * wip commit * message explaining check * temporary change * connectionManagementService restored * Revert "connectionManagementService restored" This reverts commit 9704a63. * formatting test * editConnection promise testing * edit existing connection command added * WIP Connection Edit * disconnect added to editConnection promise * WIP on editExistingConnection * changed isEdit to true * Amir/edit connection (#10112) * Get edit connection working * Delete unused code * check for isEdit as well * connection tree test added * WIP connection management tests * comment out test to find out what's wrong * fix for one error * added note about test skipped * changed signature of saveprofile * saveprofile fixed * wrote working test * added additional test * changed message * Fixes made * fix for matcher Co-authored-by: Amir Omidi <amomidi@microsoft.com>
* remove setActive method * another place
…#10248) * - Removed PG connections in the new connection wizard - If a PG database is active sending an error - Removed refresh button #6611 * Added back the refresh * -Using getConnection only once. * -Fixed the error message -Fixed the bugs mentioned in the PR Co-authored-by: Aasim Khan <aaskhan@microsoft.com>
* reoranizing existing context menu commands * adding build/deploy/import stubs * Update error message * adding for schemaCompare
* Add Cell after current active cell * fixed error * more fixes * Addressed PR * merged with master * Fixed the tests and code * removed try catch and modified the test to check the method does not throw. Co-authored-by: Rajesh Kamath <rajkashop@hotmail.com>
* rework some code and write an inital test * fix strict * add more to standard test * add to existing workflow test * fix tests * simplify the code * add more tests * remove bad import * fix compile * fix timestampiong
* bump sts * extend widget container * remove title * wip * refactoring * Revert "extend widget container" * showTitle option * fix properties widget error * icon column * icon and button columns * use textwithicon column * icon * refactor and filter * context menu * refactor * tests * fix hygiene * tests * comments
* swapping vscode calls for apiwrapper for testability * Adding mainController tests * Adding unit tests for input validation * Adding project controller tests, reorganizing error handling * Removing commented-out code
* Add real MIAA connection string values * Add MiaaModel and fix some strings * Remove unused var * Test to print env vars * Add tests * fixes
* enable tests in canary pipeline * fix bat file
* Format XML on serialization * Updating licences * Adding missing line in package.json * tabs to spaces * Updating yarn.lock file
* fix declarative table issue * check data * comments * comments and more fixes
* Stop double tree data provider registrations * revert book finding logic, leverage currentBook
* Introduced ApiWrapper for testability * Added tests for coverage of utils.ts * Addressed comments
* add database references to same itemgroup * show database references in tree * update tests * refresh project tree after a reference is added * addressing comments * add validation * Add test * sort imports
* add web compile step * add missing file * add to release * add container
* change icon locations * fix icon location
* dud space * removal of dud space * using OS line separator Co-authored-by: Benjin Dubishar <Benjin@Benjins-MBP.T-mobile.com>
* Add close button * Change the localize key
* Remove context menu items from files * More changes for context menu and adding test * change to initiate build
* convert system database references from SSDT * remove empty ItemGroup if no other database references * fix baseline files * also update sqlproj if system database references were added in SSDT since the sqlproj got updated with ADS imports * undo change * move updating system db references out of updateProjectForRoundTrip() * update test to have an already updated system db ref * add clean target after merge from master * add await * addressing comments
* intital test code * added tests * remove commented code * fix for failing tests * reuse exported enum * changes to address comments * add back onVisibility highlight * port highlight fix from Chris * fix tests
* Adding reading SqlCmdVars from project file * organized apiWrapper, added calls * Adding test to confirm deployment/script gen profiles with sqlcmd vars
* Add CodeQL Analysis workflow * Fix path
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello!
The upstream repository
Microsoft/sqlopsstudio@masterhas some new changes that aren't in this fork. So, here they are, ready to be merged! 🎉If this pull request can be merged without conflict, you can publish your software with these new changes. Otherwise, fix any merge conflicts by clicking the
Resolve Conflictsbutton.If you like Backstroke, consider donating to help us pay for infrastructure here. Backstroke is a completely open source project that's free to use, but we survive on sponsorships and donations. Thanks for your support! Help out Backstroke.
Created by Backstroke (I'm a bot!)