search

stackcollision / GardenConquest

A new conquest game mode for Space Engineers
6 stars 7 forks source link

GridEnforcer checkClassAllowed() and checkRules() have two different contracts #11

Closed zrisher closed 9 years ago

zrisher commented 9 years ago

Currently:

GridEnforcer.checkClassAllowed() returns True if the passed grid follows current rules. GridEnforcer.checkRules() returns False if the passed grid follows current rules.

I think this is a bit confusing. My preference would be to have checkRules() return True if all rules are being followed.

This is very low priority.

zrisher commented 9 years ago

Via 105122a4ca9a72bf6edff5a1460a4dcf542ace04