bugfix: add "to yml" command

[LoicRiegel]

Description

This fixes #15240, which can be closed after merge.

User-Facing Changes

• user get now use to yml -> exactly the same as to yaml

Tests + Formatting

Cargo fmt and clippy 🆗
I added a test in the only place I could find where to yaml was already tested.

I didn't see the save.rs::convert_to_extension function tested anywhere, but maybe I missed it.

After Submitting

Not sure this needs an update on the documentation ❓ What do you suggest?

[sholderbach]

Thanks!

I was thinking if we could get by with a .nu version of this as part of the std prelude. But supporting all the flags and documentation like this may still be kind of messy. So I am fine with the Rust-duplication alias.

[LoicRiegel]

Thanks!

I was thinking if we could get by with a .nu version of this as part of the std prelude. But supporting all the flags and documentation like this may still be kind of messy. So I am fine with the Rust-duplication alias.

I thought the duplication was okay because that's already the case for FromYaml and FromYml, but if there's a better approach I'm okay to change

[LoicRiegel]

Not sure this needs an update on the documentation ❓ What do you suggest?

Also what's your take on this?

[sholderbach]

I thought the duplication was okay because that's already the case for FromYaml and FromYml, but if there's a better approach I'm okay to change

Yeah fine based on precedent as well. Let's go with this approach for now

Not sure this needs an update on the documentation ❓ What do you suggest?

Also what's your take on this?

I think it suffices that the command will appear in the autogenerated command help.
We may want to improve the docs around this open/save mapping to to ... and from ... at some point.
But for this change I think you don't need to change anything in the permanent docs. Just free free to add it to the release notes as a new command with a brief example that saving with .yml ending works now.

• Release notes for 0.103.0 nushell.github.io#1783

[LoicRiegel]

Just free free to add it to the release notes as a new command with a brief example that saving with .yml ending works now.

• Release notes for 0.103.0 nushell.github.io#1783

Sure! How does it work, do I just push a new commit on the release-notes-0.103.0 branch?

[sholderbach]

If you edit it in the web interface it will probably ask you to open a PR. If you start your changes locally, you will have to set the target branch of the PR to release-notes-0.103.0.

[LoicRiegel]

Hi @sholderbach I did that PR for the release note but I think nobody saw it
I saw the contribution mentioned in the weekly blog, though

[sholderbach]

Thanks! Sorry that I didn't get around to it. Nice user-friendly writeup!