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 assert/verify/enforce separation #289

Open
mihails-strasuns-sociomantic opened this issue Oct 6, 2017 · 4 comments
Open

Revisit assert/verify/enforce separation #289

mihails-strasuns-sociomantic opened this issue Oct 6, 2017 · 4 comments

Comments

@mihails-strasuns-sociomantic
Copy link
Contributor

#212 resulted in a very straightforward migration where all assert places got changed to verify/test without any detailed consideration. This unblocks further D2 build deployment but in the long term we would need something finer tuned.

We should revisit all modules either one by one or by package and check what can be changed back to assert and what should really be an enforce with proper exception type. Eventually -release should be able to remove all non-essential checks.

@mathias-lang-sociomantic
Copy link
Contributor

This unblocks further D2 build deployment but in the long term we would need something finer tuned.

Does it ? For the dhtnode, don't we need to do the same in swarm and dhtproto ?

@gavin-norman-sociomantic

Yes, we need to do the same process in the other libraries. From ocean's side, though, things are complete.

@mihails-strasuns-sociomantic
Copy link
Contributor Author

Right, I only speak in context of ocean as it is ocean issue :)

@leandro-lucarella-sociomantic
Copy link
Contributor

Maybe a global issue in backend could help track the global progress (or a GitHub project).

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

No branches or pull requests

4 participants