Skip to content

FIX: ruamel.yaml's safe_load() has been removed#9

Open
npinto wants to merge 1 commit into
IRT-Open-Source:masterfrom
npinto:patch-1
Open

FIX: ruamel.yaml's safe_load() has been removed#9
npinto wants to merge 1 commit into
IRT-Open-Source:masterfrom
npinto:patch-1

Conversation

@npinto
Copy link
Copy Markdown

@npinto npinto commented Jan 2, 2024

@tomjnixon
Copy link
Copy Markdown
Contributor

tomjnixon commented Jan 4, 2024

The next release of EAR will switch from ruamel.yaml to pyyaml, see ebu/ebu_adm_renderer@b9ca3b7

I think it would be worth doing the same here. ruaml.yaml was originally chosen in the EAR for its ability to generate commented yaml, but that is no longer used.

You could even use ear.compatibility.load_yaml to avoid future problems like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants