Skip to content

Update custom-domains.mdx#365

Open
SU-Cloudsmith wants to merge 2 commits intomainfrom
SU-Cloudsmith-patch-3
Open

Update custom-domains.mdx#365
SU-Cloudsmith wants to merge 2 commits intomainfrom
SU-Cloudsmith-patch-3

Conversation

@SU-Cloudsmith
Copy link
Collaborator

Changing the 3 column layout to 2 column as per conversation in Slack with @jackgibson1

https://cloudsmith-io.slack.com/archives/C03RCPME8J3/p1772455542446799?thread_ts=1772452606.221399&cid=C03RCPME8J3

Needs a review to make sure the information is still correct

Changing the 3 column layout to 2 column as per conversation in Slack with @jackgibson1 

https://cloudsmith-io.slack.com/archives/C03RCPME8J3/p1772455542446799?thread_ts=1772452606.221399&cid=C03RCPME8J3

Needs a review to make sure the information is still correct
Copilot AI review requested due to automatic review settings March 10, 2026 17:18
@vercel
Copy link

vercel bot commented Mar 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cloudsmith-docs Ready Ready Preview, Comment Mar 11, 2026 9:34am

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Custom domains documentation to simplify the package support matrix from a 3-column (Download / Native Uploads / Native Downloads) layout to a 2-column (Download / Native support) layout.

Changes:

  • Replaces the 4-column package support table with a 3-column table (collapsing native upload/download into “Native support”).
  • Updates per-package entries in the table to reflect the new schema.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +38 to +48
| Cargo (Rust) | ✅ | ❌ |
| Chocolatey | - | ✅ |
| Cocoapods | ✅ | ❌ |
| Composer | ✅ | ❌ |
| Conan | - | ✅ |
| Conda | ✅ | ❌ |
| CRAN | ✅ | ❌ |
| Dart | - | ✅ |
| Debian | ✅ | ❌ |
| Docker | - | ✅ |
| Go | ✅ | ❌ |
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

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

The new "Native support" column contradicts the doc text above: formats called out as API-based/native (e.g., Cargo and Conan) are marked ❌ here. If "Native support" means “uses native API domains”, these rows should be ✅ (or otherwise rename/clarify the column to match the intended meaning).

Suggested change
| Cargo (Rust) || |
| Chocolatey | - ||
| Cocoapods |||
| Composer |||
| Conan | - ||
| Conda || |
| CRAN |||
| Dart | - ||
| Debian |||
| Docker | - ||
| Go || |
| Cargo (Rust) || |
| Chocolatey | - ||
| Cocoapods |||
| Composer |||
| Conan | - ||
| Conda || |
| CRAN |||
| Dart | - ||
| Debian |||
| Docker | - ||
| Go || |

Copilot uses AI. Check for mistakes.
Comment on lines +47 to +51
| Docker | - | ✅ |
| Go | ✅ | ❌ |
| Gradle | ✅ | ❌ |
| Helm | ✅ | ❌ |
| Hex | - | ✅ |
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

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

Several API-based formats appear to be incorrectly marked as lacking native support (e.g., Docker/Go/Gradle/Hex). Please update these values (or adjust the column definition) so the table matches the behavior described in the “API-based (Native) package formats” paragraph above.

Copilot uses AI. Check for mistakes.
| NuGet | - | ✅ |
| PowerShell | - | ✅ |
| Python | ✅ | ❌ |
| Raw | ✅ | |
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

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

More API-based formats are marked as not having native support (npm/NuGet/PowerShell/Python), which conflicts with the earlier description. Also, "Raw" previously had “not applicable” for native; consider using “-” (or another explicit N/A marker) rather than ❌ if native domains don’t apply to that package type.

Suggested change
| Raw || |
| Raw || - |

Copilot uses AI. Check for mistakes.
| RPM | ✅ | ❌ |
| Ruby | - | ✅ |
| sbt | ✅ | ❌ |
| Swift | ✅ | |
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

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

Table row formatting is inconsistent here (missing space before the trailing pipe). Please normalize spacing so markdown table rendering is consistent across rows.

Copilot uses AI. Check for mistakes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Swift has it's own API

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed spacing and adding ✅ for Native API column

| Package Type | Download | Native support |
|:--- |:--- |:--- |
| Alpine | ✅ | ❌ |
| Cargo (Rust) | ✅ | ❌ |
Copy link
Contributor

Choose a reason for hiding this comment

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

Cargo has it's own Native API

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated Native API column to ✅ for Cargo

| Cocoapods | ✅ | ❌ |
| Composer | ✅ | ❌ |
| Conan | - | ✅ |
| Conda | ✅ | ❌ |
Copy link
Contributor

Choose a reason for hiding this comment

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

Conda has it's own Native API

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated Conda Native API column to ✅

| Dart | - | ✅ |
| Debian | ✅ | ❌ |
| Docker | - | ✅ |
| Go | ✅ | ❌ |
Copy link
Contributor

Choose a reason for hiding this comment

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

Go has it's own Native API

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated Go Native API to ✅

| Helm | ✅ | ❌ |
| Hex | - | ✅ |
| LuaRocks | ✅ | ❌ |
| Maven | ✅ | ❌ |
Copy link
Contributor

Choose a reason for hiding this comment

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

Technically Maven has it's own native API too (which uploads occur on). Downloads are redirected to the DL domain from it too.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think with this one, if the way we set up the Custom Domains will differ to the rest of the formats with Native API support we should avoid labelling as such, even if it's technically supported.

Native API support being termed that a customer could set up both Upload and Download routes

Copy link
Contributor

Choose a reason for hiding this comment

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

yep this makes sense!

| RPM | ✅ | ❌ |
| Ruby | - | ✅ |
| sbt | ✅ | ❌ |
| Swift | ✅ | |
Copy link
Contributor

Choose a reason for hiding this comment

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

Swift has it's own API

| Ruby | - | ✅ |
| sbt | ✅ | ❌ |
| Swift | ✅ | |
| Terraform | ✅ | ❌ |
Copy link
Contributor

Choose a reason for hiding this comment

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

Terraform has it's own API

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated Swift and Terraform Native API support to ✅

| Cargo (Rust) | ✅ | ❌ |
| Chocolatey | - | ✅ |
| Cocoapods | ✅ | ❌ |
| Composer | ✅ | ❌ |
Copy link
Contributor

Choose a reason for hiding this comment

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

Composer now has it's own Native API

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated Composer Native API support to ✅

Updated as per @jackgibson1 's feedback
@BartoszBlizniak
Copy link
Member

For easier readability, perhaps we should change the table slightly to something like this:

Package Type Download Native API Support (Download/Upload)
Alpine -
Cargo (Rust) -
Cocoapods -
Composer -
Conan -
Conda -
CRAN -
Dart -
Debian -
Docker -
Go -
Helm -
Hex -
LuaRocks -
Maven (SBT, Gradle) Native Upload Only
npm -
NuGet (PowerShell, Chocolatey) -
Python -
Raw -
RPM -
Ruby -
Swift -
Terraform -
Unity -
Vagrant -

@BartoszBlizniak
Copy link
Member

Once this change goes in, we should also ensure that https://help.cloudsmith.io/docs/custom-domains is updated too - I can see both have different contents which might be a bit misleading.

Copy link
Member

@BartoszBlizniak BartoszBlizniak left a comment

Choose a reason for hiding this comment

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

Pre-approving

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants