Skip to content

Fix subset of linting rules - #4119

Open
smokestacklightnin wants to merge 17 commits into
tensorflow:masterfrom
smokestacklightnin:ci/linting/first-rules
Open

Fix subset of linting rules#4119
smokestacklightnin wants to merge 17 commits into
tensorflow:masterfrom
smokestacklightnin:ci/linting/first-rules

Conversation

@smokestacklightnin

@smokestacklightnin smokestacklightnin commented Aug 9, 2025

Copy link
Copy Markdown
Member

Supersedes #2252

Applies pre-commit with Ruff rules from #2251

The description from #2252 applies here:

This PR fixes a bunch of existing lint issues that were automatically fixed by ruff --fix, in preparation for merging #2251. I've ignored a bunch of rule violations, but marked them in ruff.toml so that they can be systematically re-enabled and the violations fixed one by one. The vast majority of these changes are:

  • Removing useless whitespace at the end of lines
  • Making sure the end of file contains a newline
  • Removing unnecessary python 2/3 compatibility __future__ imports
  • Import sorting
  • Replacing calls to format() with f-strings, which are faster and more expressive
@smokestacklightnin
smokestacklightnin marked this pull request as ready for review August 9, 2025 07:10
@aktech aktech mentioned this pull request Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant