New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Small improvement of `ts_utils.get_gitignore_spec` #13797

Open

Avasam wants to merge 1 commit into python:main from Avasam:small-improvement-of-ts_utils.get_gitignore_spec

Collaborator

Avasam commented Apr 4, 2025

#13795 (comment)

There is a specialized class, pathspec.GitIgnoreSpec, which more closely
implements the behavior of gitignore. This uses GitWildMatchPattern
pattern by default and handles some edge cases differently from the generic
PathSpec class. GitIgnoreSpec can be used without specifying the pattern
factory::

spec = pathspec.GitIgnoreSpec.from_lines(spec_text.splitlines())

#13795 (comment)

Could also omit the call to .readlines(). GitIgnoreSpec.from_lines takes an iterable of lines, so passing the file i/o object directly would work. Not sure if there's any reason to prefer one way or another, though


          Small improvement of ts_utils.get_gitignore_spec

02c9c64

Avasam mentioned this pull request

Enable Ruff flake8-use-pathlib (PTH) #13795

Draft

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet