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

The fuzz directory #4497

Closed
oleg-alexandrov opened this issue Apr 24, 2022 · 2 comments · Fixed by #4501
Closed

The fuzz directory #4497

oleg-alexandrov opened this issue Apr 24, 2022 · 2 comments · Fixed by #4501
Labels
Techdebt Technical dept which needs to get solved.

Comments

@oleg-alexandrov
Copy link

The top-level directory has a directory named "fuzz" which has just one single file, with no explanation for it. That looks like some kind of test code. Perhaps the repository would be cleaner if that directory was removed, and the file in question moved either to the source directory or to the test directory.

@oleg-alexandrov oleg-alexandrov added the Techdebt Technical dept which needs to get solved. label Apr 24, 2022
kimkulling added a commit that referenced this issue Apr 27, 2022
@kimkulling
Copy link
Member

Good point, I am improving the documentation for this folder.

Thanks for the hint!

@oleg-alexandrov
Copy link
Author

Thank you. It still does not make sense that a single test is kept in "fuzz", in the base directory, rather than together with the rest of the tests.

If you start getting a few more like this then the organization will become messy. But I appreciate you documenting this. This is a nice lib, and I am just picking at details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Techdebt Technical dept which needs to get solved.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants