Skip to content
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

Update FAQ script to include note on correctness of spelling mistake #654

Merged

Conversation

theoriginalbit
Copy link
Contributor

@theoriginalbit theoriginalbit commented Oct 18, 2024

Motivation

Addressing #653 (comment)

Modifications

I added a comment to the bash script provided in the FAQ to note the correctness of the spelling mistake of validation.

I also quickly went through all the documentation and fixed the few spelling mistakes I found.

Result

Better documentation for adopters, and their own review processes.

Test Plan

N/A

@theoriginalbit theoriginalbit force-pushed the fix/add-note-about-validation-typo branch from 782351e to 730a609 Compare October 18, 2024 01:06
Copy link
Contributor

@czechboy0 czechboy0 left a comment

Choose a reason for hiding this comment

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

Thank you!

@czechboy0 czechboy0 enabled auto-merge (squash) October 18, 2024 05:02
@czechboy0 czechboy0 merged commit ecdaa49 into apple:main Oct 18, 2024
26 checks passed
@theoriginalbit theoriginalbit deleted the fix/add-note-about-validation-typo branch October 18, 2024 05:39
@czechboy0 czechboy0 added the semver/none No version bump required. label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants