[GitHub] incubator-ariatosca pull request #207: ARIA-1 Parser test suite

[aviyoop]

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

    https://github.com/apache/incubator-ariatosca/pull/207#discussion_r151402533
  
    --- Diff: aria/parser/reading/reader.py ---
    @@ -28,16 +28,9 @@ def __init__(self, context, location, loader):
     
         def load(self):
             with OpenClose(self.loader) as loader:
    --- End diff --
    
    Couldn't you just made the `Loader` class to be able to be a context 
manager? What was the need to add another layer of abstraction?


---