Skip to content

[5.7] Cherry-pick DocC documentation updates. #4319

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

Merged

Conversation

zkiraly
Copy link
Contributor

@zkiraly zkiraly commented Apr 23, 2022

Cherry picked the new DocC documentation from main.

Motivation:

There is a desire that the DocC documentation be available in the 5.7 release.

Modifications:

I cherry picked these two commits from main:
911556a originally from PR #4276
5d9202f originally from PR #4277

Then I resolved a couple of merge conflicts in Sources/PackageDescription/PackageDependency.swift

All additions and changes are in the Sources/PackageDescription directory.

Result:

There were no code changes, only DocC comment additions and changes, including in the PackageDescription documentation catalog.

zkiraly and others added 2 commits April 22, 2022 20:31
…#4277)

* Part 2 of the documentation update.

* Adopted some of the comments from the companion PR swiftlang#4276.

* A few minor edits to the DocC documentation comments.

* More minor edits.

* Another minor edit.

* Removed some unwanted blank-space.

* Changed "apply" to 3rd person present tense.

* Removed blank-space.

* Fixed a typo/

* Added back the Since docs that were deleted.

Co-authored-by: Zsolt Kiraly <[email protected]>
@zkiraly
Copy link
Contributor Author

zkiraly commented Apr 23, 2022

@swift-ci please smoke test.

@abertelrud abertelrud added the 5.7 label Apr 23, 2022
@tomerd
Copy link
Contributor

tomerd commented Apr 23, 2022

@swift-ci smoke test

@tomerd tomerd merged commit bdba2ac into swiftlang:release/5.7 Apr 23, 2022
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.

3 participants