-
Notifications
You must be signed in to change notification settings - Fork 347
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
CI: fetch packages info if server tests fail #2783
Conversation
Thanks for making a pull request to Elyra! To try out this branch on binder, follow this link: |
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.
LGTM! Very nice work, thank you for your contribution and awesome turnaround time!
General question: currently the output is sorted by package name, which makes perfect sense if one wants to quickly check if one of the 'usual suspects' (packages that have broken Elyra in the past) has changed:
However, since we are also running automated daily tests (e.g. https://github.com/elyra-ai/elyra/actions/runs/2478594976) where probably sorting the output by the most recent change date (in descending order) might be a bit more useful. Would be interested in hearing what other think. |
We've discussed this in todays's dev meeting and decided to leave the report output as is for now. |
Hi, this closes #2777.
I used
pip list --outdated
. So basically, this script is only necessary because of the 'past X days' requirement.see my (other) experimental branch for demo run that fails on purpose: https://github.com/tal66/elyra/actions/runs/2477060776