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

Adjust examples in date commands #12055

Merged
merged 1 commit into from Mar 3, 2024
Merged

Conversation

robjwells
Copy link
Contributor

Hello! This is my first PR to nushell, as I was looking at things for #5066. The usage text for the date commands seemed fine to me, so this is just a bit of a tidy up of the examples, mostly the description text.

Description

  • Remove two incorrect examples for date to-record and date to-table where nothing was piped in (which causes an error in actual use).

  • Fix misleading descriptions in date to-timezone which erroneously referred to Hawaii's time zone.

  • Standardise on "time zone" in written descriptions.

  • Generally tidy up example descriptions and improve consistency.

User-Facing Changes

Only in related help text showing examples.

- Remove two incorrect examples for `date to-record` and `date to-table`
  where nothing was piped in (which causes an error in actual use).

- Fix misleading descriptions in `date to-timezone` which erroneously
  referred to Hawaii's time zone.

- Standardise on "time zone" in written descriptions.

- Generally tidy up example descriptions and improve consistency.
@robjwells
Copy link
Contributor Author

The force-push was just me rebasing the branch on upstream/main. No other changes.

@fdncred fdncred merged commit 71aacf5 into nushell:main Mar 3, 2024
19 checks passed
@fdncred
Copy link
Collaborator

fdncred commented Mar 3, 2024

Thanks

@hustcer hustcer added this to the v0.91.0 milestone Mar 4, 2024
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

3 participants