New Rule: Duplicate Property

[Snugug]

As Per SCSS Lint

[bgriffith]

Will take a look at this one today.

[bgriffith]

Any opinions on whether this should trigger when using both long hand and short hand properties?

Should the following trigger an error?

.foo {
  margin-left: 10px;
  padding: 0;
  margin: 0;
}

[DanPurdy]

I reckon it should because you are technically duplicating it there it's just sort of hidden from you, in that instance your margin-left value is being overwritten?

margin: {
  margin-top: 0,
  margin-right: 0,
  margin-bottom: 0,
  margin-left: 0
}

[bgriffith]

I've checked out the way SCSS Lint works, and they don't cause an error as technically they are different properties. Maybe this could be a different rule?

property-overwrite or something similar ? Thoughts?

[DanPurdy]

Makes sense to me, maybe that would actaully be preferable so we can keep the rules fairly granular.

[bgriffith]

Since this is only ever disallowing (as you wouldn't ever be enforcing duplicate properties), I think it should follow the current convention for 'disallows' and call it no-duplicate-property