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

style: add .style.yapf to control yapf styling of Python code #2709

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

rkuester
Copy link
Contributor

@rkuester rkuester commented Oct 1, 2024

Add a .style.yapf file so yapf can be used to style Python code without
passing the project's style via command line option. Remove the
corresponding patch to pigweed's call to yapf, used by CI, and instead
let it too rely on .style.yapf. Remove the developer documentation's
instruction to use the command line option.

BUG=description

Add a .style.yapf file so yapf can be used to style Python code without
passing the project's style via command line option. Remove the
corresponding patch to pigweed's call to yapf, used by CI, and instead
let it too rely on .style.yapf. Remove the developer documentation's
instruction to use the command line option.
@rkuester rkuester requested a review from a team as a code owner October 1, 2024 21:55
@rkuester rkuester added the ci:run label Oct 1, 2024
@TFLM-bot TFLM-bot removed the ci:run label Oct 1, 2024
@rkuester rkuester requested a review from rascani October 1, 2024 22:11
@rkuester
Copy link
Contributor Author

rkuester commented Oct 2, 2024

Some Mergify step failed.

@ddavis-2015
Copy link
Member

@Mergifyio refresh

Copy link
Contributor

mergify bot commented Oct 2, 2024

refresh

❌ Command disallowed due to command restrictions in the Mergify configuration.

  • any of:
    • sender = {{author}}
    • sender-permission >= write

@rkuester
Copy link
Contributor Author

rkuester commented Oct 2, 2024

@Mergifyio refresh

Copy link
Contributor

mergify bot commented Oct 2, 2024

refresh

✅ Pull request refreshed

@mergify mergify bot merged commit b3967a9 into tensorflow:main Oct 2, 2024
72 checks passed
@rkuester rkuester deleted the feat-yapf-style-config branch October 4, 2024 14:07
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.

4 participants