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

remove extraneous args to use_ok #8

Merged
merged 1 commit into from Jul 22, 2023

Conversation

haarg
Copy link
Contributor

@haarg haarg commented Jul 22, 2023

Additional arguments to use_ok are not a test name, but arguments passed to the module's import method. Newer versions of perl will throw errors on these unhandled arguments.

Additional arguments to use_ok are not a test name, but arguments passed to the module's import method. Newer versions of perl will throw errors on these unhandled arguments.
@briandfoy briandfoy self-assigned this Jul 22, 2023
@briandfoy briandfoy added Type: test bug an existing test is broken Status: accepted the fix is accepted Priority: low get to this whenever labels Jul 22, 2023
@briandfoy
Copy link
Owner

Thanks, it looks like we need to fix up some other things too.

@briandfoy briandfoy merged commit 00988fd into briandfoy:master Jul 22, 2023
@briandfoy briandfoy removed the Priority: low get to this whenever label Jul 22, 2023
@haarg haarg deleted the fix-use_ok-args branch July 31, 2023 02:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: accepted the fix is accepted Type: test bug an existing test is broken
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants