Skip to content

SNOW-2331878: auto release branch preparation#3764

Merged
sfc-gh-aling merged 9 commits into
mainfrom
aling-auto-release-update
Sep 13, 2025
Merged

SNOW-2331878: auto release branch preparation#3764
sfc-gh-aling merged 9 commits into
mainfrom
aling-auto-release-update

Conversation

@sfc-gh-aling

Copy link
Copy Markdown
Collaborator
  1. Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    Fixes SNOW-NNNNNNN

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
      • If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing
    • I am adding new logging messages
    • I am adding a new telemetry message
    • I am adding new credentials
    • I am adding a new dependency
    • If this is a new feature/behavior, I'm adding the Local Testing parity changes.
    • I acknowledge that I have ensured my changes to be thread-safe. Follow the link for more information: Thread-safe Developer Guidelines
    • If adding any arguments to public Snowpark APIs or creating new public Snowpark APIs, I acknowledge that I have ensured my changes include AST support. Follow the link for more information: AST Support Guidelines
  3. Please describe how your code solves the related issue.

    Please write a short description of how your code change solves the related issue.

Comment thread scripts/release/prepare_release_branch.py
Comment thread scripts/release/readme.md Outdated
- Updates `src/snowflake/snowpark/version.py` with the new VERSION tuple
- Updates `recipe/meta.yaml` with the new version string
4. **Updates Changelog**: Replaces the first version entry in `CHANGELOG.md` with the new version and today's date
5. **Updates Test Files**: Updates all `.test` and `.test.DISABLED` files in `tests/ast/data/` with the correct `client_version` format

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@sfc-gh-heshah , how long would it take to implement a python based unparser?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I would wager at most one week. If you take a look at the server-side Snowpark Unparser for guidance on output format, and the server-side Snowpark decoder for how to work with protobufs in Python, they're both extremely straightforward.

Comment thread scripts/release/prepare_release_branch.py Outdated
def check_snowpark_directory():
"""Check if we're in a snowpark-python directory"""
current_dir = os.getcwd()
if "snowpark-python" not in current_dir:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have my own repository root named snowpandas, though it's not a big deal if I need to temporarily edit the script before running it. Not sure what a more robust solution is (maybe checking git remote get-url origin matches this repo, then comparing git rev-parse --show-toplevel to the cwd?)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

feel free to contribute to this script!

Comment thread scripts/release/prepare_release_branch.py Outdated
@sfc-gh-aling sfc-gh-aling force-pushed the aling-auto-release-update branch 2 times, most recently from 9a76e50 to cba7cc4 Compare September 12, 2025 20:51
Comment thread scripts/release/prepare_release_branch.py
Comment thread scripts/release/prepare_release_branch.py Outdated
sfc-gh-aling and others added 2 commits September 12, 2025 14:13
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
@sfc-gh-aling sfc-gh-aling merged commit 5d94c7d into main Sep 13, 2025
63 of 68 checks passed
@sfc-gh-aling sfc-gh-aling deleted the aling-auto-release-update branch September 13, 2025 01:00
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants