-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
New fixes in the CI #1260
Comments
Hey I can take this up. |
I am not the maintainer or the repo, but I suggest you work on the changes. They should not take more than an hour (see the changes I made earlier). |
Hi @bhima2001 , Assigned. Thanks for contributing to DiceDB. |
@kitarp29 Hey, Is there any way I can test the workflow locally? Thanks |
Make a fork of this project on your account.
If satisfied make a PR with the main branch of your fork |
As discussed here: #1116 (comment)
I am opening a new issue for these two changes in the CI
Please make it open for all, since it's a simple change for beginners. If no one does it in a week or so.
I will make a PR against it
cc: @JyotinderSingh
The text was updated successfully, but these errors were encountered: