The test case descriptions for leap do not appear to make grammatical sense. A previous PR was made to change the description on one of the test cases, but the other test cases look to use “in” instead of “is” and I can’t seem to wrap my head around why.
I’ve created a PR to improve the consistency of the wording across the test cases.
Some of those changes are clear improvements in my opinion.
But others might guide the students too much into the right direction, for example:
- "description": "year divisible by 4, not divisible by 100 in leap year",
+ "description": "A year divisible by 4 and not divisible by 100 is a leap year",```
I think the “and” is too much of a “nudge” towards the popular one-liner that uses a logical-AND and logical-OR operator. But that might just be a minority opinion.