tliron commented on a change in pull request #214: ARIA-422 Parsing CSARs is
broken
URL:
https://github.com/apache/incubator-ariatosca/pull/214#discussion_r155181279
##########
File path: aria/utils/yaml.py
##########
@@ -48,3 +48,8 @@
finally:
from ruamel import yaml
# pylint: disable=unused-import
+
+
+# Enables support for writing python unicode class with yaml. This is a
Temporary fix which
+# should be replaced by a proper usage of ruamel.
Review comment:
Please fix this comment to specifically mention
[ARIA-429](https://issues.apache.org/jira/browse/ARIA-429), the follow up. Also
please use a capital letter for "Python", "Unicode", and all uppercase for
"YAML". Also not need to capitalize "Temporary".
Yes, this is very petty. :)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
