Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Resolve the issue: prefer single quote strings #301

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rshiue
Copy link

@rshiue rshiue commented Sep 12, 2024

What does this pull request do?

In order to prefer single quote to make consistent with Tensorflow codebase, replace double-quote with single-quote in Python files.

Fixes #279

How did you test this change?

Run pytest and pre-commit hooks(style checks)

How did you document this change?

No system behaviors got changed at all.
Just code style changes.

Before submitting

Before submitting a pull request, please be sure to do the following:

  • Read the How to Contribute guide if this is your first contribution.
  • Open an issue or discussion topic to discuss this change.
  • Write new tests if applicable.
  • Update documentation if applicable.

Copy link

google-cla bot commented Sep 12, 2024

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prefer single quote strings model-card-toolkit
1 participant