-
Notifications
You must be signed in to change notification settings - Fork 72
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: Update workflows to use latest versions of the actions #285
Conversation
Hmm, openSUSE builds also got broken (missing |
Thank you, @vt-alt! I assume this is not to be merged yet? |
I filed issue into GH checkout/action but they did not reply. This mean we will be not able to retain
ps. I said before 'it misses git' but I was incorrectly identified problem att. |
I would be OK with it if there were a better reason to do it, but removing a currently working test just to silence a warning does not feel like a good tradeoff. Do I understand this correctly? What do you think? |
Them we shall not merge it and postpone this until tests really fail with |
Perhaps they started to use |
This should fix the warning: The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/ Signed-off-by: Vitaly Chikunov <[email protected]>
Can be merged now I think. |
This should quiet the warning: