Skip to content

run black normalizations and wrappers #1061

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

Merged
merged 3 commits into from
Feb 10, 2020
Merged

Conversation

autoih
Copy link
Member

@autoih autoih commented Feb 9, 2020

related to #988

@autoih autoih requested a review from a team as a code owner February 9, 2020 23:50
@seanpmorgan
Copy link
Member

So this conflicts with a few PRs. e.g. #765 #766

IMO this shouldn't be a major deal since it just requires running the pre-commit hook on those PRs. @AakashKumarNain do you mind if we merge this and then resolve formatting on that PR?

@AakashKumarNain
Copy link
Member

@seanpmorgan I am fine with merging this first. In fact, I need to make changes to that PR, so not an issue on my side.

Copy link
Member

@seanpmorgan seanpmorgan left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@seanpmorgan seanpmorgan merged commit 4890e6e into tensorflow:master Feb 10, 2020
@autoih autoih deleted the layers branch February 11, 2020 19:09
jrruijli pushed a commit to jrruijli/addons that referenced this pull request Dec 23, 2020
* black layers fun
* modify pyproject
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.

4 participants