nit: `should do nothing when..`