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.

Sign up for GitHub

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

Jump to bottom

Replace asserts with contraints #1800

Closed
janno42 opened this issue Aug 24, 2022 · 2 comments
Closed

Replace asserts with contraints #1800

janno42 opened this issue Aug 24, 2022 · 2 comments
Assignees
@florian-str
Labels
[C] Backend Focuses on backend implementation [P] Minor Minor priority [T] Refactoring Existing parts should become faster, more readable, or in any other way better.

Comments

@janno42
Copy link
Member

janno42 commented Aug 24, 2022

A couple of asserts/checks might be better expressed as constraints. I'm not sure there was a reason why we didn't do that.

Originally posted by @karyon in #1796 (comment)
@janno42 janno42 added [C] Backend Focuses on backend implementation [P] Minor Minor priority [T] Refactoring Existing parts should become faster, more readable, or in any other way better. labels Aug 24, 2022
@niklasmohrin niklasmohrin mentioned this issue Dec 28, 2022
Closed
@richardebeling richardebeling mentioned this issue Jan 8, 2023
Open
@karyon karyon mentioned this issue Jan 16, 2023
Open
@florian-str
Copy link
Collaborator

florian-str commented Apr 24, 2023 •
edited
Loading

Constrains as checklist:

@florian-str florian-str mentioned this issue Jun 12, 2023
Merged
@niklasmohrin
Copy link
Member

Closed by #1957

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@florian-str florian-str

Labels
[C] Backend Focuses on backend implementation [P] Minor Minor priority [T] Refactoring Existing parts should become faster, more readable, or in any other way better.
Milestone
No milestone
Development

No branches or pull requests
3 participants
@janno42 @florian-str @niklasmohrin