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

Fix lint errors in predictions_to_bigquery files. #216

Merged

Conversation

cfezequiel
Copy link
Contributor

Fixes #215 .
Minimum changes to get pre-commit checks to pass without ignoring the files (i.e. pylint: skip-file).

  • [ X ] Tests pass

@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2023

Thanks for the PR! 🚀

Instructions: Approve using /lgtm and mark for automatic merge by using /merge.

@cfezequiel
Copy link
Contributor Author

Hi @rcrowe-google , @theadactyl , I was wondering if one of you would be able to review this change so it could be merged. I don't think there are any further actions needed on my end, but if there is please let me know.

@rcrowe-google rcrowe-google removed the request for review from theadactyl February 15, 2023 23:04
@rcrowe-google
Copy link
Collaborator

/lgtm
/merge

@github-actions github-actions bot merged commit d6ae065 into tensorflow:main Feb 15, 2023
@github-actions
Copy link
Contributor

Merged with approvals from rcrowe-google - thanks for the contribution! 🎉

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.

Fix lint issues in predictions_to_bigquery component.
2 participants