ci: Clean up lint task #26233

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2210-ci-lint-🗣 changing 2 files +10 −12

MarcoFalke commented at 9:49 AM on October 3, 2022: member
- Remove unused clang-format install
- Remove workaround after bumping to jammy
- Use a PPA to pin the python version instead of having to pick from the limited system-packaged python versions

Remove clang-format from lint task

This was added to use clang-format in scripted diffs, but remained
largely unused.

fabac511c6

ci: Clean up lint task

* Remove the temporary workaround
* Use the deadsnakes PPA to easier pin the python version

faccc382f3

fanquake added the label Tests on Oct 3, 2022
fanquake commented at 10:00 AM on October 3, 2022: member

Concept ACK
MarcoFalke commented at 10:04 AM on October 3, 2022: member

CI fails :man_shrugging:
MarcoFalke closed this on Oct 3, 2022
MarcoFalke deleted the branch on Oct 3, 2022
fanquake commented at 10:07 AM on October 3, 2022: member
CI fails 🤷‍♂️

Looks like that's because:
```
- Ubuntu 20.04 (focal) Python3.5 - Python3.7, Python3.9 - Python3.11
- Ubuntu 22.04 (jammy) Python3.7 - Python3.9, Python3.11
```
If you apt-get install -y python3.7-venv it works.
bitcoin locked this on Oct 3, 2023

Contributors

Labels