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

Makes it obvious how to run mongo-hacker #218

Merged
merged 1 commit into from
Apr 4, 2022

Conversation

softwarecreations
Copy link
Contributor

@softwarecreations softwarecreations commented Apr 4, 2022

Description of changes

Adds mongo to 2 places in the README

Why

I haven't used mongo-hacker for a few years, so I forgot how to run it. For whatever reason I assumed that it runs as it's own shell app.
Reading the README, it wasn't obvious to me.
I spent about 15 mins trying to figure it out, going into ~/.nvm/versions/node/v17.2.0/lib/node_modules/mongo-hacker and trying to run the js files with node haha.
I thought maybe because it hasn't been updated since 2019 (around the time of Mongo 3.4 apparently, and now we're on Mongo 5) that it was probably old and broken.
I was getting print() not defined, and _isWindows not defined etc. i tried hacking the JS and adding in const print = console.log; and const _isWindows = () => false; etc hahaha
Then finally I checked the make file and noticed it updates .mongorc then I looked at .mongorc etc and realized it probably runs inside mongo

I think 2 lines would save many people the trouble of wondering how to run it.

@TylerBrock
Copy link
Owner

Ok sure, thanks!

@TylerBrock TylerBrock merged commit e23ff0b into TylerBrock:master Apr 4, 2022
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