-
Notifications
You must be signed in to change notification settings - Fork 93
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
Added multi-variant support #93
Added multi-variant support #93
Conversation
e1b7376
to
f0d9157
Compare
Result of the new bashbrew script
|
@BlackthornYugen let me know if you want me to demo the new scripts |
That'd be great! Would you be free for a call on discord or something this weekend? |
@BlackthornYugen what is your availability? Im usually free after the kjds are in bed (I noticed we are in the samer timezone) |
I should have some time Saturday morning or maybe Sunday evening. I might be able to do something later this evening if that works. |
I can tonight after 10 if it's not too late. It wouldn't be very long |
That should work. 🙂 |
b738207
to
b634e96
Compare
@LaurentGoderre thanks for the demo. Looks great! As discussed you said you would be able to make one more change on Monday and then we can merge. 😄 |
b634e96
to
0d42f49
Compare
Yes, I have to figure out why the variant list is duplicated in some versions |
0d42f49
to
83cc647
Compare
@BlackthornYugen this is ready for final review. |
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.
Looks good, would you be able to add a short blurb to the README.md to say when to run each script?
Is it ok we add it to a CONTRIBUTING.md instead of readme like here https://github.com/nodejs/docker-node/blob/main/CONTRIBUTING.md. |
That'd be great! Just wanted to make sure it was documented somewhere. |
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.
Looks good. Thanks @LaurentGoderre!
Recreated using ./generate-stackbrew-library.sh from mongo-express/mongo-express-docker#93
TODO