Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix 765 - Update to ruamel.yaml 0.17, use new API #806

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix 765 - Update to ruamel.yaml 0.17, use new API #806

wants to merge 2 commits into from
+32 −61

Commits on Aug 25, 2022

  1. Set ruamel.yaml 0.17.2 as minimum 

    0.17.0 drops Python 2.7 support, and sets 3.5 as the minimum. Adds a
deprecation warning for dump() and load(), used in codec.py. This is
followed by rapid releases to 0.17.2 to fix packaging issues.
    @jwhitlock
    jwhitlock committed Aug 25, 2022
    Configuration menu
    Copy the full SHA
    2af7a52 View commit details
    Browse the repository at this point in the history

  2. Implement ruamel.yaml 0.15.0 API 

    Convert from `yaml.load` and `yaml.dump` to a YAML instance, and
prefer features in that library.

* SaneYamlDumper becomes SaneYamlRepresenter, based on SafeRepresenter
* increase_indent is dropped in favor of calling indent()
* represent_odict is dropped in favor of using represent_dict with
  setting sort_base_mapping_on_output to FalseA
* Drop block style for bytes
* Encode to binary after processing, rather than setting on YAML
  instance
    @jwhitlock
    jwhitlock committed Aug 25, 2022
    Configuration menu
    Copy the full SHA
    56748d3 View commit details
    Browse the repository at this point in the history