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.

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

Extensive Nu Map Cleanup #1449

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Extensive Nu Map Cleanup #1449

merged 2 commits into from
Jun 17, 2024

Conversation

NotTheDr01ds
Copy link
Contributor

Pretty extensive cleanup on aisle 6, err, Nu map from other shells and domain specific languages:

  • Formatted all commands in Markdown
  • Linked to online help for all Nushell commands
  • Replaced/fixed deprecated/removed commands (calc, nth, sum)
  • Disambiguated and fixed range (command) and range (type)
  • Bash printf added as a string formatter
  • Keybindings added for exit and clear
  • PowerShell NewItem -ItemType Directory added for mkdir
  • PowerShell Get-Command added for which
  • $env.FOO = "bar" example added for Nushell
  • Fixed incorrect Bash export statement
  • ls --du added for du
  • $nu.default-config-dir added for config
  • And more

@fdncred
Copy link
Collaborator

fdncred commented Jun 17, 2024

Nice cleanup. I'm not sure it's really possible to have this list complete. Like uname is now a thing where it wasn't when I first created this. There are a lot of str commands that aren't mentioned, which is fine. It's just kind of a hard thing to keep this evergreen.

size was renamed str stats - so that should probably be in there still.

@NotTheDr01ds
Copy link
Contributor Author

I'm not sure it's really possible to have this list complete.

Agreed, especially as I start looking at the Python (and other languages) mappings.

Like uname is now a thing where it wasn't when I first created this.

Ah, right - I did know that. I'll fix that when I get to a stopping point on the Language mapping.

size was renamed str stats - so that should probably be in there still.

Thanks - I'll update that one as well.

There are a lot of str commands that aren't mentioned, which is fine.

On the bright side, I was able to link to the main str command, so at least they are represented there.

@NotTheDr01ds
Copy link
Contributor Author

Oh, and I came back here to add some "formatting notes" as well ...

  • I tried to match Markdown column widths, but if one got too wide for a particular row, I just fixed/aligned it in the next column.
  • Because of wrapping ugliness, I've added some hardcoded <br />'s and even (since &nbsp; can't be used inside backticks), some hardcoded nonbreaking-spaces (character code 0160). Hopefully this doesn't cause issues ...

@fdncred
Copy link
Collaborator

fdncred commented Jun 17, 2024

Ping me once you're ready on this one too.

@NotTheDr01ds
Copy link
Contributor Author

Ping me once you're ready on this one too.

Should be good to go - Thanks!

@fdncred fdncred merged commit 27212af into nushell:main Jun 17, 2024
2 checks passed
@fdncred
Copy link
Collaborator

fdncred commented Jun 17, 2024

Thanks

@NotTheDr01ds NotTheDr01ds deleted the nushell-map branch June 17, 2024 17:36
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.

None yet

2 participants