ci: Use pyenv for the lint task #26637

pull miles170 wants to merge 1 commits into bitcoin:master from miles170:ci-use-pyenv-for-lint changing 2 files +20 −11

miles170 commented at 6:32 AM on December 5, 2022: contributor
Closes #26548.
1. upgrade Ubuntu Bionic to Jammy
2. use Pyenv to install Python 3.6.5 (The Python version is determined by the .python-version file in the project root)
3. cache PYENV_ROOT to skip compiling Python source code after the first run.
ci: Use pyenv for the lint task 4576281529
DrahtBot commented at 6:32 AM on December 5, 2022: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
DrahtBot added the label Tests on Dec 5, 2022
fanquake commented at 7:26 AM on December 5, 2022: member

Thanks. Note that this is already being worked on in #26581.
miles170 closed this on Dec 5, 2022
maflcko commented at 9:34 AM on December 5, 2022: member

@fanquake Are you sure? It looks like there hasn't been any activity for a week, despite feedback
miles170 commented at 9:36 AM on December 5, 2022: contributor

I can reopen the PR if necessary.
fanquake commented at 9:37 AM on December 5, 2022: member

I'm sure that there is already a PR open working on the same issue. If that one turns out to be abandoned, we can re open this. I don't think we need competing PRs open (especially for issues like this), to further disperse reviewer attention / time.
maflcko referenced this in commit 01ec5308bf on Jan 17, 2023
sidhujag referenced this in commit c4990322a2 on Jan 17, 2023
bitcoin locked this on Dec 14, 2023

Contributors

Labels

Linked (view graph)

#26548 ci: Use pyenv for the lint task

ci: Use pyenv for the lint task #26637

Reviews