-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat: Replace workflows deprecated methods #929
feat: Replace workflows deprecated methods #929
Conversation
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.
The changes here don't seem to follow the docs mentioned in the related issue:
81c97a2
to
7474466
Compare
@arboleya -> should be good now |
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.
Thanks @tibi77, LGTM!
@arboleya could you have another look? |
4d46b22
to
1a6165b
Compare
1a6165b
to
6dcdf07
Compare
I don t have access to merge this, could someone do this :) |
Should fix #928