Skip to content

Conversation

Anemy
Copy link
Member

@Anemy Anemy commented Sep 24, 2020

0.0.1-alpha.19 -> ^0.4.0

Using the new future mongosh.

Fixed a couple lints warns and updated vscode stubs with new variables in the interface.
Added acorn plugins to the webpack externals since webpack was using its version of acorn and overriding the ones the shell api is pulling in resulting in an error. *Didn't investigate deeper if its something we could update in mongosh, will do that a little now.
Here's where the acorn plugins are pulled in: https://github.com/mongodb-js/mongosh/blob/master/packages/async-rewriter/src/await.js#L9

@Anemy Anemy requested a review from alenakhineika September 24, 2020 14:05
@Anemy
Copy link
Member Author

Anemy commented Sep 24, 2020

hmmm in the built .vsix the built language server is not resolving the acorn plugins. Looking into and fixing before merging. Probably will wait to release till monday then.

@Anemy Anemy removed the request for review from alenakhineika September 28, 2020 14:42
@Anemy
Copy link
Member Author

Anemy commented Sep 28, 2020

Looks like it something we need to change on mongosh to make sure we use the acorn dependencies correctly, we can pull in a new version of mongosh once we merge in this pr and release: mongodb-js/mongosh#344

(Tests expected to fail)

@Anemy Anemy merged commit 4c199c8 into master Oct 1, 2020
@Anemy Anemy deleted the VSCODE-172/bump-mongosh-version-to-0.4.0 branch October 1, 2020 12:17
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