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

Documentation - Typos & Consistency Fixes #186

Merged
merged 1 commit into from
Jun 29, 2023
Merged

Conversation

joey-gm
Copy link
Contributor

@joey-gm joey-gm commented Jun 27, 2023

Some typo & consistency fixes for consideration.

Checklist

  • I've run tests to see all new and existing tests pass
    6 RSA key tests failed, but they are unrelated to the changes in this commit.
  • I've followed the code style of the rest of the project
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary

If you've made changes to gyb files

n.a.

Motivation:

Update documentation.

Modifications:

Only documentation has been updated. No code changes.

Copy link
Collaborator

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

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

Thanks for this! We don't accept patches to the vendored copy of BoringSSL, can you remove those?

@joey-gm
Copy link
Contributor Author

joey-gm commented Jun 27, 2023

Thanks for this! We don't accept patches to the vendored copy of BoringSSL, can you remove those?

Reverted changes in BoringSSL and squashed the commits.

Copy link
Collaborator

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

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

Thanks!

@Lukasa Lukasa added the semver/none No version bump required. label Jun 29, 2023
@Lukasa
Copy link
Collaborator

Lukasa commented Jun 29, 2023

@swift-server-bot test this please

@Lukasa Lukasa enabled auto-merge (squash) June 29, 2023 09:10
@Lukasa Lukasa merged commit cba358d into apple:main Jun 29, 2023
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