Tina Müller (tinita)

Results 261 comments of Tina Müller (tinita)

I was curious and looked into https://github.com/vubiostat/r-yaml/commit/276825d5b74bc202784324d1f986a9f09d878a94 and only see `sequence_style` mentioned, but also mappings can be in flow style, so they should also be affected. e.g. a flow mapping...

> Maybe two parameters, `default_seq_flow` and `default_map_flow`? I don't think that is necessary. What I mean is that you don't pass the parameter to the `yaml_mapping_start_event_initialize`, only to the `yaml_sequence_start_event_initialize`.

> I thought that's what this patch does, I don't see it in the changeset getting passed to yaml_mapping_start_event_initialize. And I suggested that it *should*, but with a different name,...

> What I was suggesting is to allow independent control of both. It doesn't make sense to emit all sequences in flow and all mappings in block style. everything under...

`k2:` is a valid alias name in YAML (unfortunately). PyYAML and libyaml implemented this wrong, although I can understand the decision to not support colons in alias names. Therefor, a...

Maybe it should be configurable if loading objects should be enabled or not, similar to `yaml_load_code`.

> It looks from reading the PyYAML source that YAML dicts are always presented as a Python `Ordereddict` Can you link to the code you are referring to and clarify...

There seems to be a problem with codecov. I retriggered the test (already a second time)

I hope it's ok that I pushed some minor improvements. The `assert_script_sudo_test` didn't do anything different regarding the given `$is_serial_terminal` parameter (just did the same test twice with a different...

See https://progress.opensuse.org/issues/110142#note-3 why moving to https://github.com/os-autoinst/os-autoinst-common did not work.