search

reposense / RepoSense

Contribution analysis tool for Git repositories
https://reposense.org
MIT License
251 stars 154 forks source link

GitHub Actions failing due to pip error #2249

Closed damithc closed 1 month ago

damithc commented 1 month ago

The error:

Run pip install requests
error: externally-managed-environment
...

Example https://github.com/nus-cs2113-AY2425S1/ip-dashboard/actions/runs/11285488693/job/31388669590

damithc commented 1 month ago

https://github.com/nus-cs2113-AY2425S1/tp-dashboard/actions is affected but https://github.com/nus-cs2103-AY2425S1/tp-dashboard/actions is not. hmm.... 🤔

gok99 commented 1 month ago

It looks like the breaking CI is using Ubuntu 24.04 instead of 22.04, which has configurations to prevent apt and pip from intefering with one another. Seems like the updated latest ubuntu runner isn't fully deployed.

We can probably pin the runner to ubuntu-22.04 instead of ubuntu-latest for now.

sikai00 commented 1 month ago

Beyond pinning to ubuntu-22.04 as a temporary fix, we can also look into adding a step to use venv to resolve the root issue which was introduced in PEP 668.

Can take reference from the multiple fixes that other projects have done, e.g., https://github.com/fcitx/mozc/commit/e8d25a2bee8ba5a7ac0c176cf8a8caef05663ddc#diff-1ee5360cc67f8ec7ed8a3ca75252c859e3acded65ab5e6d4688bcb8585b3e7efR29

gok99 commented 1 month ago

@damithc I've pinned publish-RepoSense to ubuntu-22.04 for the time being - could you try merging it?

@sikai00 This would be good to do, together with adding ubuntu-24.04 to our CI runners for the RepoSense repo. Could you create issues for these?

damithc commented 1 month ago

@damithc I've pinned publish-RepoSense to ubuntu-22.04 for the time being - could you try merging it?

Yup, that worked. Thanks @gok99 for the quick fix.