Skip to content

Use the correct snowflake.com domains#13363

Merged
lukasmasuch merged 1 commit intodevelopfrom
use-correct-snowflake-domains
Dec 18, 2025
Merged

Use the correct snowflake.com domains#13363
lukasmasuch merged 1 commit intodevelopfrom
use-correct-snowflake-domains

Conversation

@lukasmasuch
Copy link
Copy Markdown
Collaborator

@lukasmasuch lukasmasuch commented Dec 15, 2025

Describe your changes

Use snowflake.com instead of snowflake.cn domains when linking to the snowflake documentation.

GitHub Issue Link (if applicable)

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

Copilot AI review requested due to automatic review settings December 15, 2025 16:00
@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Dec 15, 2025 •
edited
Loading

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 15, 2025 •
edited
Loading

✅ PR preview is ready!

Name Link
📦 Wheel file https://core-previews.s3-us-west-2.amazonaws.com/pr-13363/streamlit-1.52.1-py3-none-any.whl
📦 @streamlit/component-v2-lib Download from artifacts
🕹️ Preview app pr-13363.streamlit.app (☁️ Deploy here if not accessible)

@lukasmasuch lukasmasuch mentioned this pull request Dec 15, 2025
Closed
2 tasks
Copilot AI reviewed Dec 15, 2025
View reviewed changes
Copy link
Copy Markdown
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@lukasmasuch lukasmasuch added type:docs Requests for changes to docs (will be rerouted to docs repo) security-assessment-completed impact:users PR changes affect end users change:docs PR contains documentation change and removed type:docs Requests for changes to docs (will be rerouted to docs repo) labels Dec 15, 2025
sfc-gh-bnisco
sfc-gh-bnisco approved these changes Dec 18, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@sfc-gh-bnisco sfc-gh-bnisco left a comment

Choose a reason for hiding this comment

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

As a potential follow-up, we may want to enforce expected domain names via a script/lint rule.

@lukasmasuch lukasmasuch merged commit c560bf7 into develop Dec 18, 2025
70 of 74 checks passed
@lukasmasuch lukasmasuch deleted the use-correct-snowflake-domains branch December 18, 2025 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@sfc-gh-bnisco sfc-gh-bnisco sfc-gh-bnisco approved these changes

Assignees

No one assigned

Labels

change:docs PR contains documentation change impact:users PR changes affect end users

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Snowflake documentation link is the .cn

3 participants

@lukasmasuch @sfc-gh-bnisco