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.

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

[release/1.28.1] Disable python format checking #14600

Closed

Conversation

hseok-oh
Copy link
Contributor

@hseok-oh hseok-oh commented Feb 3, 2025

It disables python format checking because it is not working on yapf 0.22.0.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh [email protected]


Relataed issue: #14596

It disables python format checking because it is not working on yapf 0.22.0.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <[email protected]>
@hseok-oh
Copy link
Contributor Author

hseok-oh commented Feb 3, 2025

If we don't need more patch on release/1.28.1, close this PR.

@seanshpark
Copy link
Contributor

Frankly speaking, I don't know what is going on with 1.28.1 release branch.
Are we going to release this version?

@hseok-oh
Copy link
Contributor Author

hseok-oh commented Feb 3, 2025

@seanshpark I also don't know and I never heard about 1.28.1 release.

@hseok-oh
Copy link
Contributor Author

hseok-oh commented Feb 3, 2025

@seanshpark It looks like about #14548

@hseok-oh hseok-oh marked this pull request as draft February 3, 2025 10:38
@dahlinPL
Copy link
Contributor

@seanshpark It looks like about #14548

Hi, we gave up with asking to release 1.28.1, even if it will be useful for us. I believe we will wait until 1.30.0 will be released and try to bump ONE dependency in our project then. Eventually if there will be some issues we will ask to introduce some improvements in 1.30.1. So to summarize if this PR is related with our request to release 1.28.1 it can be safely closed. Thanks :)

@hseok-oh hseok-oh closed this Mar 24, 2025
@hseok-oh hseok-oh deleted the disable_ptyhon_check branch March 24, 2025 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants