You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ariatosca.apache.org by "Tal Liron (JIRA)" <ji...@apache.org> on 2017/12/06 09:25:00 UTC

[jira] [Updated] (ARIA-429) Change our usage of ruamel.yaml to the new API

     [ https://issues.apache.org/jira/browse/ARIA-429?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tal Liron updated ARIA-429:
---------------------------
    Description: 
The new API is recommended for various reasons mentioned here: http://yaml.readthedocs.io/en/latest/api.html#reason-for-api-change

But more specifically, it intrinsically solves the problem for which we created a workaround n ARIA-422.

Also, it seems the new API automatically tries to initialize the {{CSafeLoader}} (see {{parsing/reading/yaml.py}}). We need to verify that this is true.

If all is good, we need to move all our uses of ruamel to this new API and centralize them in our {{utils/yaml.py}}.

  was:
The new API is recommended for various reasons mentioned here: http://yaml.readthedocs.io/en/latest/api.html#reason-for-api-change

But more specifically, it intrinsically solves the problem for which we created a workaround n ARIA-422.

Also, it seems the new API automatically tries to initialize the CSafeLoader (see `parsing/reading/yaml.py`). We need to verify that this is true.

If all is good, we need to move all our uses of ruamel to this new API and centralize them in our `utils/yaml.py`.


> Change our usage of ruamel.yaml to the new API
> ----------------------------------------------
>
>                 Key: ARIA-429
>                 URL: https://issues.apache.org/jira/browse/ARIA-429
>             Project: AriaTosca
>          Issue Type: Story
>            Reporter: Tal Liron
>
> The new API is recommended for various reasons mentioned here: http://yaml.readthedocs.io/en/latest/api.html#reason-for-api-change
> But more specifically, it intrinsically solves the problem for which we created a workaround n ARIA-422.
> Also, it seems the new API automatically tries to initialize the {{CSafeLoader}} (see {{parsing/reading/yaml.py}}). We need to verify that this is true.
> If all is good, we need to move all our uses of ruamel to this new API and centralize them in our {{utils/yaml.py}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)