Github user tliron commented on a diff in the pull request:

    https://github.com/apache/incubator-ariatosca/pull/207#discussion_r153014351
  
    --- Diff: aria/parser/presentation/presentation.py ---
    @@ -199,6 +199,9 @@ class Presentation(PresentationBase):
         """
     
         def _validate(self, context):
    +        if (not 
context.presentation.configuration.get('validate_normative', True)) \
    +            and self._get_extension('normative'):
    +            return
    --- End diff --
    
    That user should never be using our ARIA `_extensions` and setting 
`normative: true`. These `_extensions` are entirely for internal use by our 
project. If they really want to use this feature, it is up to them to make sure 
the types are valid. (We test for this as part of our tox test suite by 
disabling this flag.)


---

Reply via email to