Skip to content
New issue

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

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

Already on GitHub? # to your account

rename --os/-o to --platform/-p #390

Closed
wants to merge 1 commit into from

Conversation

pixelcmtd
Copy link
Member

this has already caused a lot of confusion (tldr-pages/tldr#6869, tldr-pages/tldr#9252 (review), …), is not standard (https://github.com/tldr-pages/tldr/blob/9ac1ffd5ca55f977da63d954788be5c463def51a/CLIENT-SPECIFICATION.md#arguments) and has already been fixed in other clients (tealdeer-rs/tealdeer#209)

  • Code compiles correctly
  • Created tests, if possible
  • All tests passing (npm run test:all)
  • Extended the README / documentation, if necessary

@pixelcmtd
Copy link
Member Author

I just discovered #337 -.-

@pixelcmtd pixelcmtd closed this Oct 24, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant