Skip to content

Conversation

@yedpodtrzitko
Copy link
Contributor

@yedpodtrzitko yedpodtrzitko commented Sep 13, 2024

aimed against the branch of #499

@CyanVoxel CyanVoxel merged commit 16dd634 into TagStudioDev:ruff-changes Sep 13, 2024
@CyanVoxel CyanVoxel added Type: Refactor Code that needs to be restructured or cleaned up Type: CI Continuous Integration / workflows labels Sep 13, 2024
CyanVoxel added a commit that referenced this pull request Sep 13, 2024
* ci: update ruff linter config

- Set line length to 100
- Enforce Google-style docstrings
- Lint docstrings and imports

* ci(ruff): exclude missing docstring warnings

* ci(ruff): exclude docstring checks from tests dir

* fix(ruff): change top level linter setting

Fix Ruff warning: `warning: The top-level linter settings are deprecated in favour of their counterparts in the `lint` section. Please update the following options in `pyproject.toml`:
  - 'per-file-ignores' -> 'lint.per-file-ignores'`.

* chore: format with ruff

* add `.git-blame-ignore-revs`

* ci(ruff): add E402 and F541 checks

* ci(ruff): add FBT003 check (#500)

* ci(ruff): add T20 check

* ci(ruff)!: add N check, refactor method names

* ci(ruff): add E501 check, refactor strings

Much commented-out code is removed in this commit.

* update `.git-blame-ignore.revs`

---------

Co-authored-by: yed <yedpodtrzitko@users.noreply.github.com>
@yedpodtrzitko yedpodtrzitko deleted the yed/ruff-fbt003 branch November 18, 2024 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: CI Continuous Integration / workflows Type: Refactor Code that needs to be restructured or cleaned up

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants