Skip to content

Conversation

@Gold240sx
Copy link

@Gold240sx Gold240sx commented Oct 16, 2025

Added my SPM Repo to integrate file uploads and deletions into native Swift applications.

Summary by CodeRabbit

  • Documentation
    • Added documentation for a new native Swift backend adapter enabling UploadThing integration in Swift projects.

@changeset-bot
Copy link

changeset-bot bot commented Oct 16, 2025

⚠️ No Changeset found

Latest commit: d9bec26

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Oct 16, 2025

@Gold240sx is attempting to deploy a commit to the Ping Labs Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 16, 2025

Walkthrough

README.md was updated to include a new entry describing a fully native Swift Package Manager backend adapter for UploadThing integration with Swift projects. This is a documentation-only change with no code modifications.

Changes

Cohort / File(s) Summary
Documentation Update
README.md
Added entry describing the new SPM package for UploadThing's native Swift backend adapter

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title “Update README.md” is generic and does not highlight the specific addition of the Swift Package Manager integration, leaving the main change unclear. Please revise the title to reference the new Swift Package Manager adapter entry, for example “Add Swift Package Manager adapter to README,” so it clearly conveys the primary change.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
README.md (1)

36-37: Clarify third‑party status and tweak phrasing/casing

To avoid implying official maintenance and to align style/casing, consider marking this as community‑maintained and tighten the copy.

Proposed edit:

- [Swift Package Manager (SPM) Package](https://github.com/Gold240sx/UploadThing-Swift) -
-  A fully native backend adapter for incorporating uploadthing in your Swift projects.
+ [Swift Package (SPM) — third‑party](https://github.com/Gold240sx/UploadThing-Swift) -
+  A community‑maintained native Swift package for integrating UploadThing uploads and deletions into Swift projects.

Optional: If you maintain a “Community packages/integrations” subsection, moving this bullet there may improve clarity. Please confirm desired placement.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b1d759e and d9bec26.

📒 Files selected for processing (1)
  • README.md (1 hunks)

Copy link
Collaborator

@markflorkowski markflorkowski left a comment

Choose a reason for hiding this comment

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

I think this makes more sense in the docs (/docs/src/app/(docs)/uploading-files/page.mdx) thank in the readme. There is mention of the community Python adapter, probably somewhere around there.

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.

2 participants