-
Notifications
You must be signed in to change notification settings - Fork 199
fix bin/up with mongo change #351
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
base: master
Are you sure you want to change the base?
Conversation
with overleaf#270 the prefix of the configuration shouldn't be in the return value. On a new checkout I ran into ``` --------------------- ERROR ----------------------- Invalid MONGO_VERSION: MONGO_VERSION=6.0 MONGO_VERSION must start with the actual major version of mongo, followed by a dot. Example: MONGO_IMAGE=my.dockerhub.com/custom-mongo MONGO_VERSION=6.0-custom --------------------- ERROR ----------------------- ``` removing `\1` fixed it.
@mserranom can I borrow your eyes on this? I think #270 changed the mongo config and the prefix shouldn't be returned by |
Thanks @darvid7, would you mind posting the steps to reproduce, including the problematic inputs and their outputs vs the expected outputs? |
For sure! this is how I ran into the issue
I was following the steps in https://github.com/overleaf/toolkit/blob/master/doc/quick-start-guide.md With the removal of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm unable to reproduce. What I'm a bit puzzled about is this log:
When I set an invalid version in
I only get the actual value of the variable:
Would you mind sharing the output of |
Looks like it's the same problem reported in #287:
|
Description
fix
bin/up
setupwith #270 the prefix of the configuration shouldn't be in the return value.
On a new checkout I ran into
removing
\1
fixed it.Related issues / Pull Requests
Contributor Agreement