-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix the failing CI github workflow #8
base: develop
Are you sure you want to change the base?
Fix the failing CI github workflow #8
Conversation
This was causing error while installing python requirements. It's conflicting with `actions/setup-python@v4`
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.
please mention reason why it was added in first place, if possible add reference
I had generated the github workflow file using ChatGPT and tested it and it was working earlier. Acc. to my findings, the workflow fails when the base runner image is Acc. to this github issue, they have removed some of the packages and changed versions of few packages in ubuntu 24 image. A combination of base runner image Using |
why this @v4
|
I had picked it up from a code sample from an article and not from the official docs. Therefore it's an older (v4) version |
.github/workflows/test.yml
Outdated
with: | ||
python-version: '3.12' | ||
python-version: '3.12.7' |
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.
why we can't go with something like 3.12.* like we do have in others repo even in frontend repos
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.
Changed it to 3.12.*
Date: 20 Dec 2024
Developer Name: Samarpan Harit
Issue Ticket Number
Description
actions/setup-python@v4
actions/setup-python@v4
leading to this issue.Documentation Updated?
Under Feature Flag
Database Changes
Breaking Changes
Development Tested?
Screenshots
The workflow successfully running for this PR is the proof of it's working
Success workflow execution for this PR
Test Coverage
Not applicable
Additional Notes