Generated by 🚫 Danger

I added some unit tests. I also changed a bit the contract of @NoAttribute:

• Attributes from Node and AbstractNode are never suppressed. I don't know what might go wrong if we do suppress them.
• The scope INHERITED suppresses the attributes inherited from supertypes
• The scope ALL suppresses the attributes defined in the class + INHERITED
• In any case, if an attribute accessor is overridden in a concrete class C, then it is only suppressed if either
  1. the accessor has its own NoAttribute annotation
  2. C has a @NoAttribute(scope = ALL).

This last point means that if you can override the method, then you can expose it as an attribute again.

@oowekyala anything holding this from being merged?

@jsotuyod nope, i might have forgotten about it