Skip to content

yield configuration path in configuration path fixture#117

Merged
altendky merged 2 commits into
developmentfrom
yield_from_configuration_path_fixture
Apr 17, 2021
Merged

yield configuration path in configuration path fixture#117
altendky merged 2 commits into
developmentfrom
yield_from_configuration_path_fixture

Conversation

@altendky

Copy link
Copy Markdown
Collaborator

https://docs.python.org/3.9/library/importlib.html#importlib.resources.path

If needed, importlib.resources.path() will write the resources to a temporary location. This will be cleaned up after exiting the context manager. Using yield leaves the fixture 'in' the context manager until the fixture is cleaned up.

https://docs.python.org/3.9/library/importlib.html#importlib.resources.path

If needed, importlib.resources.path() will write the resources to a temporary location.  This will be cleaned up after exiting the context manager.  Using yield leaves the fixture 'in' the context manager until the fixture is cleaned up.
@altendky altendky marked this pull request as ready for review April 17, 2021 02:26

@jkbecker jkbecker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@altendky altendky merged commit 19c9670 into development Apr 17, 2021
@altendky altendky deleted the yield_from_configuration_path_fixture branch April 17, 2021 22:44
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.

2 participants