-
Notifications
You must be signed in to change notification settings - Fork 17
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(tooling): clients scripts and Docker image #43
Conversation
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.
docker ftw
…i-clients-automation into chore/APIC-225/clients-script
8f8fe12
to
f51e16d
Compare
"docker:clean": "docker stop dev; docker rm -f dev; docker image rm -f api-clients-automation", | ||
"docker:mount": "./scripts/docker/mount.sh", | ||
"docker:setup": "yarn docker:clean && yarn docker:build && yarn docker:mount", | ||
"docker": "docker exec dev yarn $*", |
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.
Is there a way to pass the signals too ? Like ctrl+c is not stopping the generation
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 did not managed to reproduce, I'm able to use ctrl+c on my side 🤔
This reverts commit 2a90ccd.
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.
It works great :)
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.
\o/
🧭 What and Why
🎟 JIRA Ticket:
Changes included:
I had to create the Docker image since I did not wanted to install MVN etc. on my machine, I've took the base made by @millotp
clients
script to build clients🧪 Test