Skip to content

Conversation

@IvanTechnicalWriter
Copy link
Contributor

@IvanTechnicalWriter IvanTechnicalWriter commented Sep 9, 2025

Summary:
Switch generic example URL in README from ucarecdn.com to demo.ucarecd.net.

Notes:
Docs-only; badges/logos/images left intact. One README change, one commit.

Resolves: DOCS-1156

Summary by CodeRabbit

  • Documentation
    • Updated the Upload API example to use a demo domain (demo.ucarecd.net) for uploaded file URLs, improving clarity for users testing uploads.
    • Only the example URL changed; all other example code remains the same.
    • No impact on runtime behavior or public APIs. No action required for existing integrations.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 9, 2025

Walkthrough

A single documentation edit in README.md updates an example upload URL from ucarecdn.com to demo.ucarecd.net. No code or API behavior changes.

Changes

Cohort / File(s) Summary
Docs
README.md
In the “Using Upload API” example, changed the uploaded file URL from https://ucarecdn.com/46528d0d-323c-42d7-beab-2fdc5e7077ba/ to https://demo.ucarecd.net/46528d0d-323c-42d7-beab-2fdc5e7077ba/.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly summarizes the main change by indicating that the README is being updated with subdomain examples for the uploadcare-swift library and includes the relevant issue key, making it concise, descriptive, and directly related to the changeset.
Description Check ✅ Passed The description succinctly summarizes the specific change to the README, notes that it is docs-only, and references the resolved ticket, making it directly relevant to the changeset without being off-topic or overly vague.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)
  • JIRA integration encountered authorization issues. Please disconnect and reconnect the integration in the CodeRabbit UI.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between c986c38 and e303e5c.

📒 Files selected for processing (1)
  • README.md (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Build and test on latest macOS
  • GitHub Check: Build and test on latest Ubuntu (5.1)
  • GitHub Check: Build and test on latest Ubuntu (5.6.3)
🔇 Additional comments (1)
README.md (1)

120-120: URL update matches new per‑project CDN subdomains.

Switch to demo.ucarecd.net aligns with Uploadcare’s July 22, 2025 subdomain rollout (<hash>.ucarecd.net/:uuid/). Legacy ucarecdn.com links remain valid for existing projects. (uploadcare.com)

If “demo” is not a guaranteed public example host, consider using a placeholder like https://<your-subdomain>.ucarecd.net/<UUID>/ to avoid dead links in the README.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch Docs-1156-libs-readme-subdomains

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.

@IvanTechnicalWriter IvanTechnicalWriter merged commit 9acc9f0 into master Sep 9, 2025
5 of 8 checks passed
@IvanTechnicalWriter IvanTechnicalWriter deleted the Docs-1156-libs-readme-subdomains branch September 9, 2025 11:09
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