Open bexsoft opened 1 week ago
Do we merge with the CI failure?
I don’t think we should. Why do we need a new console release at this time?
Do we merge with the CI failure?
Yes, This PDF issue is not a real failure, PDF is rendered correctly in the UI Locally I see an error related to pdf worker library required for this and testcafe. This needs further investigation
@bexsoft Did you regenerate the files after rebasing to the latest
master
? When I runmake release
, then all generated files are different but that may be caused due to different versions. I do think thebuild
folder should be identical and not depend on whoever runsmake release
.For that reason, I am always a fan of letting the CI/CD build the final output instead. Then it's consistent.
Yes, @ramondeklein, all files are updated with the latest master change. we use a release script to generate these files, maybe could be related
@bexsoft corresponding docs ready to merge https://github.com/minio/docs/pull/1373
Do we merge with the CI failure?