Skip to content

Fix typo, refine wording in Usage.md #8423

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
Apr 1, 2025
Merged

Fix typo, refine wording in Usage.md #8423

merged 1 commit into from
Apr 1, 2025

Conversation

MaxDesiatov
Copy link
Contributor

@MaxDesiatov MaxDesiatov commented Mar 28, 2025

Avoiding specifying -> Avoid specifying

Also reordered words in this somewhat convoluted sentence. "provided by the library in the module.modulemap" could've been read as if the library provides something in such modulemap file. But in fact, "in the module.modulemap" is related to the first part of the sentence. Moving it there makes the sentence less ambiguous.

`Avoiding specifying` -> `Avoid specifying`

Also reordered clauses in this somewhat convoluted sentence. "provided by the library in the `module.modulemap`" could've been read as the library provides something in such modulemap. But in fact, "in the `module.modulemap`" is related to the first part of the sentence.
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test macos

@MaxDesiatov
Copy link
Contributor Author

@swift-ci smoke test macos

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test windows

@bkhouri
Copy link
Contributor

bkhouri commented Apr 1, 2025

@swift-ci please test self hosted windows

@bkhouri
Copy link
Contributor

bkhouri commented Apr 1, 2025

@swift-ci please test windows

@bkhouri
Copy link
Contributor

bkhouri commented Apr 1, 2025

@swift-ci please test macOS

@MaxDesiatov MaxDesiatov merged commit fb96b98 into main Apr 1, 2025
6 checks passed
@MaxDesiatov MaxDesiatov deleted the maxd/usage-wordin branch April 1, 2025 06:17
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.

3 participants