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

Import of testing package #144

Closed
yukels opened this issue Jan 23, 2020 · 5 comments
Closed

Import of testing package #144

yukels opened this issue Jan 23, 2020 · 5 comments
Labels

Comments

@yukels
Copy link

yukels commented Jan 23, 2020

You import the testing package in the manager_test_helper.go file

As a result any program which is using the flag standard package and depends on the ladon library with produce many extra flags:

-test.bench string
        regular expression to select benchmarks to run
-test.benchmem
        print memory allocations for benchmarks
-test.benchtime duration
        approximate run time for each benchmark (default 1s)
-test.blockprofile string
        write a goroutine blocking profile to the named file after execution

Please slip the testing code from the production one

@aeneasr aeneasr added the bug label Jan 23, 2020
@aeneasr
Copy link
Member

aeneasr commented Jan 23, 2020

Wow, I did not know that that could be an issue! We should probably move this to a separate directory :) Contributions welcome!

@yukels
Copy link
Author

yukels commented Mar 4, 2020

Hi, do you plan to add the fix to the next release?

@aeneasr
Copy link
Member

aeneasr commented Mar 5, 2020

I am unfortunately very busy and do not have time to fix this, but as I said in the previous comment, you are more than welcome to create a PR to resolve this!

@yukels
Copy link
Author

yukels commented Apr 13, 2020 via email

@aeneasr
Copy link
Member

aeneasr commented Sep 4, 2021

I am closing this issue as it has not received any engagement from the community or maintainers in a long time. That does not imply that the issue has no merit. If you feel strongly about this issue

  • open a PR referencing and resolving the issue;
  • leave a comment on it and discuss ideas how you could contribute towards resolving it;
  • open a new issue with updated details and a plan on resolving the issue.

We are cleaning up issues every now and then, primarily to keep the 4000+ issues in our backlog in check and to prevent maintainer burnout. Burnout in open source maintainership is a widespread and serious issue. It can lead to severe personal and health issues as well as enabling catastrophic attack vectors.

Thank you to anyone who participated in the issue! 🙏✌️

@aeneasr aeneasr closed this as completed Sep 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants