Skip to content

Fix parameter documentation in PackageDependency.swift #7682

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
merged 1 commit into from
Jun 18, 2024

Conversation

SimplyDanny
Copy link
Contributor

  • Remove mention of nil name which is not allowed due to the parameters being non-optional.
  • Synchronize wording describing local package paths.

* Remove mention of `nil` name which is not allowed due to the parameters being non-optional.
* Synchronize wording for local package paths.
Copy link
Contributor

@MaxDesiatov MaxDesiatov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks!

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@MaxDesiatov MaxDesiatov enabled auto-merge (squash) June 18, 2024 20:43
@MaxDesiatov MaxDesiatov disabled auto-merge June 18, 2024 20:43
@MaxDesiatov MaxDesiatov changed the title Fix parameter documentation Fix parameter documentation in PackageDependency.swift Jun 18, 2024
@MaxDesiatov MaxDesiatov enabled auto-merge (squash) June 18, 2024 20:44
@MaxDesiatov MaxDesiatov merged commit 83214f0 into swiftlang:main Jun 18, 2024
5 checks passed
@SimplyDanny SimplyDanny deleted the patch-1 branch June 19, 2024 05:35
MaxDesiatov pushed a commit that referenced this pull request Jun 27, 2024
* Remove mention of `nil` name which is not allowed due to the parameters being non-optional.
* Synchronize wording describing local package paths.

(cherry picked from commit 83214f0)

# Conflicts:
#	Sources/PackageDescription/PackageDependency.swift
MaxDesiatov added a commit that referenced this pull request Jun 27, 2024
Includes these PRs cherry-picked off `main`
* #7605
* #7660
* #7667
* #7682
* #7687
* #7690
* #7684
* #7679

**Explanation**: Cherry-pick of recent NFC changes, which makes it
easier to cherry-pick actual bug fixes onto 6.0 due to the reduced
number of merge conflicts.
**Scope**: broad, includes both modules graph and llbuild-related
changes.
**Risk**: low, the test suite is passing, no functional changes are
included, and cherry-picked changes were incubated on `main` for some
time.
**Testing**: Existing automated test suite.
**Issue**: N/A
**Reviewers**: @xedin @MaxDesiatov @rauhul

---------

Co-authored-by: Pavel Yaskevich <[email protected]>
Co-authored-by: Danny Mösch <[email protected]>
Co-authored-by: Rauhul Varma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants