Skip to content

[CI] Fix PR AutoFix GitHub action#40257

Merged
sergiitk merged 5 commits intogrpc:masterfrom
sergiitk:fix/ci/pr-auto-fix
Jul 22, 2025
Merged

[CI] Fix PR AutoFix GitHub action#40257
sergiitk merged 5 commits intogrpc:masterfrom
sergiitk:fix/ci/pr-auto-fix

Conversation

@sergiitk
Copy link
Member

@sergiitk sergiitk commented Jul 18, 2025

Previously broken by requiring black version that dropped py3.8 support.
Gitignore/dockerignore changes made to prevent ruff checking virtualenv directories.

Related:

Previously broken by requiring black version that drop py3.8 support.
@sergiitk sergiitk requested a review from eugeneo July 22, 2025 22:29
pip install pyyaml mako virtualenv absl-py
sudo apt-get update
sudo apt-get install python3-dev
pip list
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this needed?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not needed, but is very useful since we don't pin the dependencies here.

@eugeneo
Copy link
Contributor

eugeneo commented Jul 22, 2025

(I approved it)

@sergiitk sergiitk merged commit 9772d15 into grpc:master Jul 22, 2025
61 checks passed
@sergiitk sergiitk deleted the fix/ci/pr-auto-fix branch July 22, 2025 23:44
asheshvidyut pushed a commit to asheshvidyut/grpc that referenced this pull request Jul 28, 2025
Previously broken by requiring black version that dropped py3.8 support.
Gitignore/dockerignore changes made to prevent ruff checking virtualenv
directories.

Related:
- grpc#40256 
- grpc#40139
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request Aug 23, 2025
Previously broken by requiring black version that dropped py3.8 support.
Gitignore/dockerignore changes made to prevent ruff checking virtualenv
directories.

Related:
- grpc#40256 
- grpc#40139
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants