Skip to content

feat(cli-repl): add support for bracketed paste in REPL MONGOSH-1909 #2328

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

Merged
merged 5 commits into from
Jan 29, 2025

Conversation

addaleax
Copy link
Collaborator

Bracketed paste allows us to receive a copy-pasted piece of mongosh as a single block, rather than interpreting it line-by-line. For now, this requires some monkey-patching of Node.js internals, so a follow-up ticket will include work to upstream support for this into Node.js core.

Bracketed paste allows us to receive a copy-pasted piece of mongosh
as a single block, rather than interpreting it line-by-line.
For now, this requires some monkey-patching of Node.js internals, so
a follow-up ticket will include work to upstream support for this
into Node.js core.
@@ -43,7 +43,7 @@
"access": "public"
},
"engines": {
"node": ">=16.15.0"
"node": ">=18.19.0"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Minimum version with nodejs/node#47150

@@ -1510,9 +1540,6 @@ describe('e2e', function () {

describe('history file', function () {
it('persists between sessions', async function () {
if (process.arch === 's390x') {
return this.skip(); // https://jira.mongodb.org/browse/MONGOSH-746
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Drive-by because this is also fixed by the env var fixup in .evergreen/setup-env.sh

@gagik
Copy link
Contributor

gagik commented Jan 28, 2025

Since this touches @mongosh/e2e-tests might be worth holding on until #2332 is merged and the temporary push-tags workflow is run just so we can have an accurate tag representing the currently published version.

@gagik
Copy link
Contributor

gagik commented Jan 28, 2025

Since this touches @mongosh/e2e-tests might be worth holding on until #2332 is merged and the temporary push-tags workflow is run just so we can have an accurate tag representing the currently published version.

Tags have been pushed 😄

@addaleax addaleax merged commit 4232a98 into main Jan 29, 2025
129 of 134 checks passed
@addaleax addaleax deleted the 1909-dev branch January 29, 2025 16:14
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.

3 participants