Conversation
AlexWaygood
left a comment
AlexWaygood
left a comment
There was a problem hiding this comment.
docs/reference/configuration.md is generated and needs to be updated over in the Ruff repo (see failing CI)
|
Oh sorry, didn't realize this also changes a non-generated file. |
| Note that a `./python` folder is automatically added to the project `root` if it exists, | ||
| and is not itself a package (i.e. does not contain an `__init__.py` file or an |
There was a problem hiding this comment.
Why do we discuss this down here, separately from the mention of ./src above? Isn't the treatment of ./python the same as the treatment of ./src? (I realize that choice predates this PR.)
There was a problem hiding this comment.
Hmm other variants of this doc seemed to suggest there were two mutually exclusive "layout modes" we detected, but the ./python addition was applicable to both.
There was a problem hiding this comment.
I don't think that is the case...
| By default, ty searches for first-party modules in the project's root directory or the `src` | ||
| directory, if present. |
There was a problem hiding this comment.
| By default, ty searches for first-party modules in the project's root directory and in `src` and `python` | |
| subdirectories, if the latter are present and are not packages (don't contain an `__init__.py` or `__init__.pyi` file). |
|
Oh ha, I had auto-merge set from before Carl had comments, and Alex dismissing his review unblocked it, whoops! |
3 participants
As of astral-sh/ruff#21817, ./tests are no longer implicitly included.