-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
fix pkg3 docs typo #28422
fix pkg3 docs typo #28422
Conversation
stdlib/Pkg/docs/src/index.md
Outdated
@@ -695,7 +695,7 @@ Testing... | |||
|
|||
Sometimes one might want to use some packages only at testing time but not | |||
enforce a dependency on them when the package is used. This is possible by | |||
adding `[extra]` dependencies and adding a a "test target" to the Project file. | |||
adding `[extras]` dependencies and adding a a "test target" to the Project file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a duplicate "a" in adding a a "test target"
in this line as well.
Also, I'd suggest formatting that bit as adding a "test" `[target]`
(or adding a `[target]` named "test"
), to be consistent with the usage of `[extras]`
earlier in the sentence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix to [extras]
. Note that the [targets]
section is plural as well, however.
@@ -695,7 +695,7 @@ Testing... | |||
|
|||
Sometimes one might want to use some packages only at testing time but not | |||
enforce a dependency on them when the package is used. This is possible by | |||
adding `[extra]` dependencies and adding a a "test target" to the Project file. | |||
adding dependencies to `[extras]` and a `test` target in `[targets]` to the Project file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"to `[targets]`", rather than "in `[targets]`"?
* Update index.md (cherry picked from commit a451801)
No description provided.