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

chore: modernize repository #1

Merged
merged 7 commits into from
Aug 7, 2024
Merged

chore: modernize repository #1

merged 7 commits into from
Aug 7, 2024

Conversation

addaleax
Copy link
Collaborator

@addaleax addaleax commented Aug 7, 2024

Updates to make sure we can keep using the socksv5 package. This is already based on the heroku fork of socksv5, and I'd plan on making the first few commits here part of a pull request to their fork (as a "take it or leave it" kind of thing – it's probably as unmaintained as the original package).

Whether this is enough for us to close https://jira.mongodb.org/browse/COMPASS-5472 or not, I'm not sure, but honestly I'd lean towards saying it is.

I've tried to keep changes somewhat minimal (i.e. no style fixes and no integration into devtools-shared) so that we can abandon this fork if at some point something better comes up.


chore: make tests pass when localhost resolves to ::1
fix: replace usage of new Buffer()
chore: add 'use strict' directives
chore(deps): update ip-address to 9.x
chore: add github workflows
chore: drop yarn, use npm instead
chore: rename package to mongodb-js prefix

package-lock.json Outdated Show resolved Hide resolved
@addaleax addaleax merged commit 66f45dd into main Aug 7, 2024
10 checks passed
@himanshusinghs himanshusinghs deleted the modernize branch August 7, 2024 10:58
addaleax added a commit to mongodb-js/devtools-shared that referenced this pull request Aug 7, 2024
addaleax added a commit to mongodb-js/devtools-shared that referenced this pull request Aug 7, 2024
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