CodeChain-io / CodeChain-io/codechain

Need an assertion policy

Ouverte
#197 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Rust
Étoiles
256
Forks
50
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Currently, we use `debug_assert` in most cases because we check the assertions in tests and we don't want to check them again in the release binary.

But it's not sustainable. Bugs will occur for any reason. It seems better to use `assert` to crash early because we have a plan to add [a bug tracker](https://github.com/CodeChain-io/codechain/issues/196).

I think we need a guideline to decide when to use `assert` or `debug_assert`.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.