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

Delete unused test fixture #7737

Merged
merged 1 commit into from
Aug 7, 2023
Merged

Delete unused test fixture #7737

merged 1 commit into from
Aug 7, 2023

Conversation

jeffwidman
Copy link
Member

@jeffwidman jeffwidman commented Aug 5, 2023

This file appears unused. It was added in
b6ae9d3 but I don't see any reference to it in that commit's changes to pip-compile spec files.

I do not see any accompanying .in file, so only this needs to be removed.

@jeffwidman jeffwidman marked this pull request as ready for review August 5, 2023 16:19
@jeffwidman jeffwidman requested a review from a team as a code owner August 5, 2023 16:19
Copy link
Member

@Nishnha Nishnha left a comment

Choose a reason for hiding this comment

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

This file appears unused. It was added in
b6ae9d3
but I don't see any reference to it in that commit's `pip-compile` spec
files.

I do not see any accompanying `.in` file, so only this needs to be
removed.
@jeffwidman jeffwidman force-pushed the delete-unused-fixture branch from 33479ed to 66c4f80 Compare August 7, 2023 16:05
@jeffwidman jeffwidman enabled auto-merge (squash) August 7, 2023 16:08
@jeffwidman jeffwidman merged commit d8a2654 into main Aug 7, 2023
@jeffwidman jeffwidman deleted the delete-unused-fixture branch August 7, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants