-
Notifications
You must be signed in to change notification settings - Fork 55
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): Reuse built extension on tests #1438
chore(ci): Reuse built extension on tests #1438
Conversation
Looks like the errors are different on different platform. Some path related issues. On windows, it's On Mac/Linux, it's |
Yes, I misunderstood the documentation of the download-artifact action, so it was downloading in the incorrect path. I pushed a commit with a fix 🙂 |
One more thing's missing 😅 even though we don't need to run |
…p-node to v4 to cache the NPM dependencies.
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.
Fantastic! It's not an easy task to update such a complex build description file.
Thank you for the contribution!
As discussed in #1419:
A better way to run CI is to build the extension and test it on different Java versions. We can leverage the artifacts already built on the
build-and-analyse
job to do that. We upload the required artifacts of the built extension and then download them to be reused on thetest-extension
job.