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

Move to organization necessitates code review and tests before wider use #3

Open
hkolbeck opened this issue Jul 1, 2023 · 0 comments

Comments

@hkolbeck
Copy link
Member

hkolbeck commented Jul 1, 2023

While this code has been manually tested a great deal, it lacks tests. This is not a simple issue, as unit testing anything that's centrally built around an RNG is far from easy.

Comprehensive testing is not necessarily possible, but some review by folks other than the author is needed before wider use. This review is in two forms: Assuring that the language definition pieces are correct and up to date, and verifying that the code is accurately expressing those definitions.

The more eyes on this the better, as this project's usage could greatly assist in ensuring the correctness of any existing or new parsers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant