-
Notifications
You must be signed in to change notification settings - Fork 384
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
Add unzip to factory #1015
Add unzip to factory #1015
Conversation
|
7eb2960
to
31836f8
Compare
31836f8
to
0fec546
Compare
@CamilleDrapier What did you mean by 'does not seem to work' in this statement?
|
Sorry @jennifer-shehane , this was a bit unclear from me! Here is what I get when I try the
The penultimate line is probably nothing serious, and can be ignored as you mentioned in cypress-io/cypress#28061 (comment) The last line, I'm unsure about it, I was assuming that there is something I'm missing when running this (and that the CI is checking properly?); but I was unable to identify what so far 🙇 |
Ah, yeah, I believe these can be ignored. |
This is an attempt to solve: #1013
This is based on the documentation in README/CONTRIBUTING, but I might have misunderstood something, please let me know if that is the case.
After making these changes, I ran the following (for some reason, the images built with 'included' does not seem to work, and I am unsure why):