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

Revisit the assertions libraries #20

Open
technosophos opened this issue Jul 19, 2019 · 1 comment
Open

Revisit the assertions libraries #20

technosophos opened this issue Jul 19, 2019 · 1 comment

Comments

@technosophos
Copy link
Member

As discussed here, I think we're all having some mixed feelings about Spectral as the library we want to use for assertion testing. It can be a little verbose in lieu of simpler assertions (like the core assert!/assert_eq! macros). Now might be a good time to decide.

/cc @jlegrone and @glyn

@glyn
Copy link

glyn commented Jul 23, 2019

There's a lot to be said for avoiding unnecessary dependencies, so I'd favour using core macros unless the code becomes a lot more verbose, in which case it might be worth adopting another library.

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

2 participants