FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Move show-fixtures code from python.py to fixtures.py by bluetech · Pull Request #11910 · pytest-dev/pytest · GitHub

Move show-fixtures code from python.py to fixtures.py - #11910

Merged
bluetech merged 1 commit into
pytest-dev:mainfrom
bluetech:move-show-fixtures
Apr 2, 2024
Merged

Move show-fixtures code from python.py to fixtures.py#11910
bluetech merged 1 commit into
pytest-dev:mainfrom
bluetech:move-show-fixtures

Conversation

bluetech commented Feb 2, 2024

Copy link
Copy Markdown
Member

It makes more sense, also, we have a long term idea of generalizing fixture support to items defined by other plugins, not just python, in which case --fixtures would definitely not be python-plugin specific.

WarrenTheRabbit commented Mar 18, 2024
edited
Loading

Copy link
Copy Markdown
Contributor

@bluetech Moving the code to fixtures.py sounds great to me, and your remark about generalising fixture support is very interesting.

I recently tried to resolve an issue with #12129. @The-Compiler floated the idea originally in #11295 and I give three justifications in my PR description (and one comment) for making the change.

But in short, I was working in python.py. Intuitively, I would have preferred to be in fixtures.py, and in fact I had to import a function from there. I was wanting to make a separate proposal to move the code into fixtures.py, or indeed to ask what the rules are around breaking out content from large files into specialised subfiles. e.g. parts of fixtures.py into show_fixtures.py, for instance.

Also, greetings to all. I hope everyone has been well. Hopefully I have now returned and can be more involved; assuming, in (my recollection of) @RonnyPfannschmidt's words, I'm not put through another bad fate compression algorithm.

It makes more sense, also, we have a long term idea of generalizing
fixture support to items defined by other plugins, not just python, in
which case `--fixtures` would definitely not be python-plugin
specific.
bluetech force-pushed the move-show-fixtures branch from d0de530 to 1188ea2 Compare April 2, 2024 20:04
bluetech merged commit b63bea9 into pytest-dev:main Apr 2, 2024
bluetech deleted the move-show-fixtures branch April 2, 2024 20:28
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL