Skip to content

Modified yaml_fwrite to preserve file insertion order#611

Open
leoechevarria wants to merge 1 commit intoolofk:mainfrom
leoechevarria:main
Open

Modified yaml_fwrite to preserve file insertion order#611
leoechevarria wants to merge 1 commit intoolofk:mainfrom
leoechevarria:main

Conversation

@leoechevarria
Copy link
Copy Markdown

Added sort_keys=False to yaml.dump call in yaml_fwrite to preserve the order of insertion of the files to be written in the .core when writing a generator (for instance for packages to be included and compiled before other sources). Supported from PyYAML >= 5.1.

@imphil
Copy link
Copy Markdown
Collaborator

imphil commented Mar 7, 2023

Thanks for your contribution! Can you add a test to show the problem you're fixing, and to ensure that it actually stays fixed?

And please add a version constraint on pyyaml if you depend on a feature that's only available from a certain version on (https://github.com/olofk/fusesoc/blob/main/setup.py#L56).

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