Skip to content

Conversation

@jerpelea
Copy link
Contributor

Summary

ci: add Python linter to check
[FIX CI] ci: use venv for check #14176
To avoid the following CI error:

This environment is externally managed
--> To install Python packages system-wide, try apt install
python3-xyz, where xyz is the package you are trying to
install.

If you wish to install a non-Debian-packaged Python package,
create a virtual environment using python3 -m venv path/to/venv.
Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
sure you have python3-full installed.

If you wish to install a non-Debian packaged Python application,
it may be easiest to use pipx install xyz, which will manage a
virtual environment for you. Make sure you have pipx installed.

See /usr/share/doc/python3.12/README.venv for more information.

Impact

Fixes current checks CI.
Testing

See the checks CI run.

casaroli and others added 2 commits October 15, 2024 17:03
To avoid the following CI error:

This environment is externally managed
--> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    If you wish to install a non-Debian packaged Python application,
    it may be easiest to use pipx install xyz, which will manage a
    virtual environment for you. Make sure you have pipx installed.

    See /usr/share/doc/python3.12/README.venv for more information.
@jerpelea jerpelea requested a review from anchao October 15, 2024 15:06
@github-actions github-actions bot added Area: CI Size: XS The size of the change in this PR is very small labels Oct 15, 2024
@xiaoxiang781216 xiaoxiang781216 merged commit 9971626 into apache:releases/12.7 Oct 15, 2024
@jerpelea jerpelea deleted the bp-14323 branch October 16, 2024 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: CI Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants