Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adhere to PEP 8 #15127

Closed
wants to merge 1 commit into from
Closed

Adhere to PEP 8 #15127

wants to merge 1 commit into from
+3 −3

Conversation

ghost
Copy link

@ghost ghost commented Dec 18, 2023 •
edited by ghost
Loading

Fix #14528

@ghost ghost self-requested a review as a code owner December 18, 2023 07:05
@ghost ghost requested review from haberman and removed request for a team December 18, 2023 07:05
@google-cla Google CLA
Copy link

google-cla bot commented Dec 18, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@ghost
Copy link
Author

ghost commented Dec 18, 2023

@hlopko @haberman Please approve workflow run.

@hlopko hlopko added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Dec 18, 2023
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Dec 18, 2023
@ghost
Copy link
Author

ghost commented Dec 22, 2023

@hlopko @haberman Is there an option to re-run one job?

2023-12-18T20:00:54.7943560Z [ 29%] Built target log_internal_check_op
2023-12-19T01:56:50.9313220Z ##[error]The operation was canceled.

copybara-service bot pushed a commit that referenced this pull request Jan 31, 2024
@copybara-github
Adhere to PEP 8 (#15127)
461234b 
Fix #14528

Closes #15127

COPYBARA_INTEGRATE_REVIEW=#15127 from vyq:main 0c1f01c
FUTURE_COPYBARA_INTEGRATE_REVIEW=#15127 from vyq:main 0c1f01c
PiperOrigin-RevId: 602865716
@copybara-service copybara-service bot mentioned this pull request Jan 31, 2024
Closed
zhangskz pushed a commit that referenced this pull request Feb 5, 2024
@zhangskz
Adhere to PEP 8 (#15127)
1c078d5 
Fix #14528

Closes #15127

COPYBARA_INTEGRATE_REVIEW=#15127 from vyq:main 0c1f01c
PiperOrigin-RevId: 603092749
@zhangskz zhangskz mentioned this pull request Feb 5, 2024
Merged
zhangskz added a commit that referenced this pull request Feb 5, 2024
@zhangskz
Adhere to PEP 8 (#15127) (#15729)
3b92f4f 
Fix #14528

Closes #15127

COPYBARA_INTEGRATE_REVIEW=#15127 from vyq:main 0c1f01c
PiperOrigin-RevId: 603092749

Co-authored-by: vyq <[email protected]>
deannagarcia pushed a commit to deannagarcia/protobuf that referenced this pull request Jun 20, 2024
@deannagarcia
Adhere to PEP 8 (protocolbuffers#15127)
885fb74 
Fix protocolbuffers#14528

Closes protocolbuffers#15127

COPYBARA_INTEGRATE_REVIEW=protocolbuffers#15127 from vyq:main 0c1f01c
PiperOrigin-RevId: 603092749
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Output _pb2.py that adheres to PEP 8
2 participants
@haberman @hlopko