-
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(ci): use docker #2525
base: main
Are you sure you want to change the base?
chore(ci): use docker #2525
Conversation
✅ Deploy Preview for api-clients-automation ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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 LOVE IT GG
65c56bb
to
d11b584
Compare
🔨 The codegen job will run at the end of the CI.Make sure your last commit does not contain generated code, it will be automatically pushed by our CI. |
this doesn't seem to be faster, I think we need to shrink the image size a lot |
If fine with it not being faster personally as long as it reduces the friction between local/CI it's still a huge improvement |
Wooo this is indeed a big improvment if it does not kills the CI duration ! |
5326587
to
d7f57dc
Compare
🧭 What and Why
Instead of initizalizing everything locally and on the CI, we can reuse the same docker image to save a lot of time.