-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
NFC: 5.9 Documentation update. #6434
Conversation
…wift-package-manager into zkiraly/5.9-documentation-update Catching up with the origin.
This is ready for engineering review. |
@swift-ci please smoke test. |
@@ -47,7 +47,7 @@ public struct Resource { | |||
|
|||
/// The rule for the resource. | |||
let rule: String | |||
|
|||
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.
nit: trailing whitespace
@@ -12,6 +12,7 @@ | |||
|
|||
/// The build configuration such as debug or release. | |||
public struct BuildConfiguration { | |||
/// The configuration of the build. Valid values are `debug` and `release`. | |||
let config: String |
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.
This should not be public, right? Will it still show up in the developer docs?
@swift-ci smoke test |
* Updated documentation for BuildSettings. * Updated documentation for PackageDescription.
Review and update the documentation for SwiftPM 5.9
Motivation:
A couple of new API were added to
PackageDescription
, and these needed new documentation, and in some cases review.Modifications:
Documentation was added or updated. Nearly all the changes are for post 5.7 API.
Result:
There will be up-to-date documentation in
PackageDescription
that conforms to the current documentation style.