Skip to content

Bump version to v3.16.0-rc1#2807

Merged
sfc-gh-jwilkowski merged 9 commits intorelease-v3.16.0from
cherrypicks-v3.16.0-rc1
Mar 17, 2026
Merged

Bump version to v3.16.0-rc1#2807
sfc-gh-jwilkowski merged 9 commits intorelease-v3.16.0from
cherrypicks-v3.16.0-rc1

Conversation

@sfc-gh-jwilkowski
Copy link
Contributor

Pre-review checklist

  • I've confirmed that instructions included in README.md are still correct after my changes in the codebase.
  • I've added or updated automated unit tests to verify correctness of my new code.
  • I've added or updated integration tests to verify correctness of my new code.
  • I've confirmed that my changes are working by executing CLI's commands manually on MacOS.
  • I've confirmed that my changes are working by executing CLI's commands manually on Windows.
  • I've confirmed that my changes are up-to-date with the target branch.
  • I've described my changes in the release notes.
  • I've described my changes in the section below.
  • I've described my changes in the documentation.

Changes description

...

sfc-gh-moczko and others added 5 commits March 10, 2026 10:19
Add protobuf>=5.29.6,<6 as an explicit dependency to resolve
HIGH severity uncontrolled recursion vulnerability in
json_format.ParseDict() (SNYK-PYTHON-PROTOBUF-15090738).

protobuf is a transitive dependency via snowflake-snowpark-python.
Version 5.29.6 is a patch release within the >=3.20,<6.34 constraint.

.... Generated with [Cortex Code](https://docs.snowflake.com/en/user-guide/cortex-code/cortex-code)

Co-authored-by: Maciej Oczko <maciej.oczko+snow@snowflake.com>
Co-authored-by: Cortex Code <noreply@snowflake.com>
@sfc-gh-jwilkowski sfc-gh-jwilkowski marked this pull request as ready for review March 10, 2026 15:41
@sfc-gh-jwilkowski sfc-gh-jwilkowski requested a review from a team as a code owner March 10, 2026 15:41
sfc-gh-olorek
sfc-gh-olorek previously approved these changes Mar 11, 2026
Copy link

@sfc-gh-olorek sfc-gh-olorek left a comment

Choose a reason for hiding this comment

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

lgtm

sfc-gh-mdrach and others added 3 commits March 17, 2026 11:25
…2805)

* fix: escape glob special chars in directory paths during stage copy

Directories with square brackets (e.g., [id], [slug]) failed during
recursive stage copy with error 'File doesn't exist: [.../[id]/*]'.

The bug occurred because [id]/* was interpreted as a glob pattern
matching 'i/*' or 'd/*' instead of the literal '[id]/*' directory.

Fix: Use glob.escape() before appending /* to directory paths. This
only affects real directories (checked via .is_dir()), so user-provided
glob patterns like 'src/[abc]' or '*.py' continue to work correctly.

Changes:
- manager.py:406,369-372: Add glob.escape() when appending wildcards
- test_stage.py: Add 4 tests for brackets and glob pattern preservation

* fix: strip trailing slashes before glob.escape() in stage put

A path like '/tmp/campaigns/[id]/' would produce '/tmp/campaigns/[id]//*'
after escaping and appending '/*'. Strip trailing slashes first so the
result is always clean (e.g. '/tmp/campaigns/[[]id]/*').

Also removes test_stage_put_preserves_user_glob_patterns_with_brackets
which re-implemented the logic under test rather than calling StageManager,
and adds two new tests: trailing slash alone and trailing slash + brackets.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
SNOW-2326969 Improved text styles to use a lighter blue instead of standard blue in DCM to increase the contrast when the black terminal background is used
Copy link

@sfc-gh-olorek sfc-gh-olorek left a comment

Choose a reason for hiding this comment

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

lgtm

@sfc-gh-jwilkowski sfc-gh-jwilkowski merged commit a19b294 into release-v3.16.0 Mar 17, 2026
33 checks passed
@sfc-gh-jwilkowski sfc-gh-jwilkowski deleted the cherrypicks-v3.16.0-rc1 branch March 17, 2026 11:00
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.

5 participants