Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Decouple test_session_completer from project level noxfile #480

Merged
merged 2 commits into from
Sep 17, 2021
Merged

Decouple test_session_completer from project level noxfile #480

merged 2 commits into from
Sep 17, 2021

Conversation

FollowTheProcess
Copy link
Collaborator

Didn't raise an issue for this because it was discussed here: #479 (comment)

This PR decouples test_session_completer from the project's actual noxfile by adding a new fake noxfile with multiple sessions under tests/resources and pointing test_session_completer at it using the keyword argument.

@theacodes theacodes merged commit e7d4f5a into wntrblm:main Sep 17, 2021
@theacodes
Copy link
Collaborator

👍🏻 nice

@FollowTheProcess FollowTheProcess deleted the decouple-noxfile-test branch September 18, 2021 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants