Closed njzjz closed 4 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 46.88%. Comparing base (
bec4f2e
) to head (f2e23a2
). Report is 1 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I leave this PR as a draft until we want to deprecate Python 3.7.
.github/workflows/pyright.yml (2)
`14-14`: Ensure the `uv` package is available in the Python package index before using `pip install uv`. --- `15-15`: Ensure that the `uv pip install --system` command is compatible with editable installs (`-e` option).ci/ssh.sh (1)
`9-9`: Ensure that the `uv` tool is properly installed and configured within the Docker container before using it for dependency management.ci/ssh_rsync.sh (1)
`13-13`: Ensure that the `uv` tool is properly installed and configured within the Docker container before using it for dependency management.ci/pbs.sh (1)
`12-12`: Ensure that the `uv` tool is properly installed and configured within the Docker container before using it for dependency management. Also, verify that the `chown -R pbsuser .` command does not inadvertently change permissions on files that should not be modified..github/workflows/test.yml (4)
`28-28`: Ensure that the `curl` command successfully installs the `uv` tool on non-Windows platforms. Consider adding error handling to ensure the installation process does not silently fail. --- `30-30`: Ensure that the `powershell` command successfully installs the `uv` tool on Windows platforms. Consider adding error handling to ensure the installation process does not silently fail. --- `32-32`: Ensure that the `uv pip install --system` command is compatible with the project's dependency management requirements. --- `35-38`: The change to use `python -m coverage` instead of `coverage` directly is a good practice as it ensures that the correct Python interpreter is used. No further action needed..github/workflows/test-bohrium.yml (2)
`23-23`: Ensure the `uv` package is available in the Python package index before using `pip install uv`. --- `24-24`: Ensure that the `uv pip install --system` command is compatible with the Bohrium environment's dependency management requirements.
Summary by CodeRabbit