Skip to content

Update custom-domains.mdx #365

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.

Sign up for GitHub

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

Open
SU-Cloudsmith wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update custom-domains.mdx #365

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f15c403
Update custom-domains.mdx
SU-Cloudsmith Mar 10, 2026
2093c3d
Update custom-domains.mdx
SU-Cloudsmith Mar 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
62 changes: 31 additions & 31 deletions src/content/workspaces/custom-domains.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,37 +32,37 @@ It is possible to have multiple download domains for a package type, however, al

Please refer to the table below to see which package will utilize Download/API Native domains.

| Package Type | Download | Native Uploads | Native Downloads |
|:--- |:--- |:--- |:--- |
| Alpine | ✅ | ❌ | ❌ |
| Cargo (Rust) | ✅ | ❌ | ✅ |
| Chocolatey | - | ✅ | ✅ |
| Cocoapods | ✅ | ❌ | ❌ |
| Composer | ✅ | ❌ | ✅ |
| Conan | - | ✅ | ✅ |
| Conda | ✅ | ❌ | ✅ |
| CRAN | ✅ | ❌ | ❌ |
| Dart | - | ✅ | ✅ |
| Debian | ✅ | ❌ | ❌ |
| Docker | - | ✅ | ✅ |
| Go | ✅ | ❌ | ✅ |
| Gradle | ✅ | ✅ | ❌ |
| Helm | ✅ | ❌ | ❌ |
| Hex | - | ✅ | ✅ |
| LuaRocks | ✅ | ❌ | ❌ |
| Maven | ✅ | ✅ | ❌ |
| npm | - | ✅ | ✅ |
| NuGet | - | ✅ | ✅ |
| PowerShell | - | ✅ | ✅ |
| Python | ✅ | ✅ | ❌ |
| Raw | ✅ | - | - |
| RPM | ✅ | ❌ | ❌ |
| Ruby | - | ✅ | ✅ |
| sbt | ✅ | ✅ | ❌ |
| Swift | ✅ | ✅ | ❌ |
| Terraform | ✅ | ❌ | ✅ |
| Unity | - | ✅ | ✅ |
| Vagrant | ✅ | ❌ | ❌ |
| Package Type | Download | Native support |
Comment thread
jackgibson1 marked this conversation as resolved.
Outdated
|:--- |:--- |:--- |
| Alpine | ✅ | ❌ |
| Cargo (Rust) | ✅ | ❌ |
Copy link
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 10, 2026

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
Copy Markdown
Contributor Author

@SU-Cloudsmith SU-Cloudsmith Mar 11, 2026

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

| Chocolatey | - | ✅ |
| Cocoapods | ✅ | ❌ |
| Composer | ✅ | ❌ |
Copy link
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 10, 2026

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
Copy Markdown
Contributor Author

@SU-Cloudsmith SU-Cloudsmith Mar 11, 2026

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 ✅

| Conan | - | ✅ |
| Conda | ✅ | ❌ |
Copy link
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 10, 2026

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
Copy Markdown
Contributor Author

@SU-Cloudsmith SU-Cloudsmith Mar 11, 2026

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 ✅

| 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.
Copy link
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 10, 2026

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
Copy Markdown
Contributor Author

@SU-Cloudsmith SU-Cloudsmith Mar 11, 2026

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 ✅

| Gradle | ✅ | ❌ |
| Helm | ✅ | ❌ |
| Hex | - | ✅ |
Comment on lines +47 to +51
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.
| LuaRocks | ✅ | ❌ |
| Maven | ✅ | ❌ |
Copy link
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 10, 2026

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
Copy Markdown
Contributor Author

@SU-Cloudsmith SU-Cloudsmith Mar 11, 2026

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
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 11, 2026

Choose a reason for hiding this comment

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

yep this makes sense!

| npm | - | ✅ |
| 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
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 10, 2026

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
Copy Markdown
Contributor Author

@SU-Cloudsmith SU-Cloudsmith Mar 11, 2026

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

| Terraform | ✅ | ❌ |
Copy link
Copy Markdown
Contributor

@jackgibson1 jackgibson1 Mar 10, 2026

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
Copy Markdown
Contributor Author

@SU-Cloudsmith SU-Cloudsmith Mar 11, 2026

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 ✅

| Unity | - | ✅ |
| Vagrant | ✅ | ❌ |

### Top Level Redirects

Expand Down