Comment on Show HN: Ellipsis – Automated PR reviews and bug fixesparentComments−hunterbrooksOP2yGood catch. The team could add this rule to their Ellipsis config file to make sure that it's always flagged: "Never use magic numbers. Always store the number in a variable and use the variable instead."Docs: https://docs.ellipsis.dev/config#add-custom-rules−azinman22yBut even that isn’t ALWAYS the case. There are times when it is appropriate to have numbers inline, as long as they’re not repeated.This is where good judgement comes in, which is difficult to encode rules for.
Comments
Good catch. The team could add this rule to their Ellipsis config file to make sure that it's always flagged: "Never use magic numbers. Always store the number in a variable and use the variable instead."
Docs: https://docs.ellipsis.dev/config#add-custom-rules
But even that isn’t ALWAYS the case. There are times when it is appropriate to have numbers inline, as long as they’re not repeated.
This is where good judgement comes in, which is difficult to encode rules for.