-
Notifications
You must be signed in to change notification settings - Fork 101
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: upgrade missed e2e upload artifact to v4 #1937
Conversation
📝 Acceptance Test Report📋 Summary✅ The rule acceptance has passed for commit 41dc2f0 📊 Notices ComparisonNew Errors (0 out of 1788 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Errors (0 out of 1788 datasets, ~0%) ✅No changes were detected due to the code change. New Warnings (0 out of 1788 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Warnings (0 out of 1788 datasets, ~0%) ✅No changes were detected due to the code change. 🛡️ Corruption Check0 out of 1788 sources (~0 %) are corrupted. ⏱️ Performance Assessment📈 Validation TimeAssess the performance in terms of seconds taken for the validation process.
📜 Memory Consumption
|
Summary:
Missed from previous #1926 work.
Out of scope
Leaving out of scope the fix of Python scripts running in ubuntu-latest. This will be solved in #1938
Expected behavior:
The upload artifact is upgraded to v4, and the E2E test passed. Successfully build link.
Please make sure these boxes are checked before submitting your pull request - thanks!
gradle test
to make sure you didn't break anything