Skip to content

Revise CONTRIBUTING.md#4

Merged
tfpauly merged 2 commits into
mainfrom
parispittman-patch-1
Jun 9, 2026
Merged

Revise CONTRIBUTING.md#4
tfpauly merged 2 commits into
mainfrom
parispittman-patch-1

Conversation

@parispittman

Copy link
Copy Markdown
Contributor

Worked on cleaning up and styling the guide with more actions and structure

Worked on cleaning up and styling the guide with more actions and structure
Comment thread CONTRIBUTING.md Outdated
* All code must conform to the [Swift API Design Guidelines](https://swift.org/documentation/api-design-guidelines/)
* Documentation is required. Please explain the "why" behind non-obvious decisions.

Given SwiftTLS's limited scope, please confirm the fix or change via pull request is within scope by opening an [Issue](https://github.com/apple/swift-tls/issues) first. The [Networking category on the Swift Forums](https://forums.swift.org/c/development/networking/129) is also a great place to discuss feature requests and larger overall project discussions.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should we have a reference to the limited scope (in the README)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yeah! that's a good idea - I'll add.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added; the README PR should be merged first since I changed the header that links: #5

thanks!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

That's merged now!

Comment thread CONTRIBUTING.md Outdated
Co-authored-by: Paris <paris_pittman@apple.com>
@tfpauly tfpauly merged commit 4c1e5be into main Jun 9, 2026
19 checks passed
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