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

Docs - Inconsistent docs for the parameter "-WorkingDirectory" in Install-ChocolateyShortcut #1234

Closed
OlafHaag opened this issue Apr 4, 2017 · 3 comments
Assignees
Labels
5 - Released Bug Documentation NO RELEASE NOTES Should not be included in the release notes - not enhancing or fixing end product. Up For Grabs
Milestone

Comments

@OlafHaag
Copy link

OlafHaag commented Apr 4, 2017

What You Are Seeing?

Inconsistent use for the parameter "-WorkingDirectory" in Install-ChocolateyShortcut helper function documentation. In Example 2 it's "-WorkDirectory", in the chocolateyInstall.ps1 that comes with choco new it's "-workDirectory".

What is Expected?

That it's always called "-WorkingDirectory" in order to not confuse me and make errors.

How Did You Get This To Happen? (Steps to Reproduce)

choco new, and looking at the docs: https://chocolatey.org/docs/helpers-install-chocolatey-shortcut, looking at the source code

@ferventcoder
Copy link
Member

Documentation bug. Up for grabs.

gep13 pushed a commit that referenced this issue Mar 11, 2019
This is a small fix to the Parameter definition of
Install-ChocolateyShortcut.ps1. This aligns $WorkingDirectory to be the
same name in the Comment Based Help and Param() block. This will also
fix the upstream documentation for the command on chocolatey.org.
gep13 added a commit that referenced this issue Mar 11, 2019
* pr1730:
  (maint) Corrected whitespace
  (GH-1234) Fix typo in Parameter definition
gep13 added a commit that referenced this issue Mar 11, 2019
* stable:
  (maint) Corrected whitespace
  (GH-1234) Fix typo in Parameter definition
  (maint) formatting
  (GH-1580) Only log when RegularOutput is enabled
  (maint) Whitespace changes
@gep13 gep13 modified the milestones: 0.10.14, 0.10.12 Mar 11, 2019
@gep13
Copy link
Member

gep13 commented Mar 11, 2019

This was merged into stable at 26ace80

@gep13
Copy link
Member

gep13 commented Mar 11, 2019

Merged into stable to be released with 0.10.12.

@gep13 gep13 closed this as completed Mar 11, 2019
@ferventcoder ferventcoder changed the title Inconsistent docs for the parameter "-WorkingDirectory" in Install-ChocolateyShortcut Docs -Inconsistent docs for the parameter "-WorkingDirectory" in Install-ChocolateyShortcut Mar 13, 2019
@ferventcoder ferventcoder added the NO RELEASE NOTES Should not be included in the release notes - not enhancing or fixing end product. label Mar 13, 2019
@ferventcoder ferventcoder changed the title Docs -Inconsistent docs for the parameter "-WorkingDirectory" in Install-ChocolateyShortcut Docs - Inconsistent docs for the parameter "-WorkingDirectory" in Install-ChocolateyShortcut Mar 13, 2019
corbob added a commit to corbob/choco that referenced this issue May 15, 2022
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf lkjdf lkj lkj lkj lkj asdfsadfasdf
lkjdf lkj lkj lkj lkj asdfsadfasdf
corbob added a commit to corbob/choco that referenced this issue May 15, 2022
No new line this time. Let's see how long we can make this line. We'll
just keep typing, and have it be long enough to wrap a few times. How
much more do you think we need? We're at > 100 beyond 72 🤷 Might
as well keep going. Maybe we'll stop at 300? Or perhaps 4? Naw, let's
just do it 200.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Released Bug Documentation NO RELEASE NOTES Should not be included in the release notes - not enhancing or fixing end product. Up For Grabs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants