Skip to content

ensure update-version.sh preserves alpha spec, add tests on version constants - #2344

Merged
rapids-bot[bot] merged 3 commits into
NVIDIA:branch-24.08from
jameslamb:more-rbb
Jun 4, 2024
Merged

ensure update-version.sh preserves alpha spec, add tests on version constants#2344
rapids-bot[bot] merged 3 commits into
NVIDIA:branch-24.08from
jameslamb:more-rbb

Conversation

@jameslamb

Copy link
Copy Markdown
Member

Description

Contributes to rapidsai/build-planning#31

Follow-up to #2331

  • ensures that update-version.sh does not remove alpha specs like ,>=0.0.0a0 in pyproject.toml and conda environment files
  • consolidates rapids-build-backend versions in dependencies.yaml
  • adds tests that __git_commit__ and __version__ constants are present and that __version__ is populated

@jameslamb jameslamb added 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 29, 2024
@jameslamb
jameslamb requested review from KyleFromNVIDIA and bdice May 29, 2024 22:55
@jameslamb
jameslamb requested review from a team as code owners May 29, 2024 22:55
@jameslamb

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot
rapids-bot Bot merged commit f690e5c into NVIDIA:branch-24.08 Jun 4, 2024
@jameslamb
jameslamb deleted the more-rbb branch June 4, 2024 14:55
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
…onstants (NVIDIA#2344)

Contributes to rapidsai/build-planning#31

Follow-up to NVIDIA#2331

* ensures that `update-version.sh` does not remove alpha specs like `,>=0.0.0a0` in `pyproject.toml` and conda environment files
* consolidates `rapids-build-backend` versions in `dependencies.yaml`
   - *since I was pushing a new commit here anyway, figured I'd take the opportunity to include that simplification recommended in NVIDIA/cudf#15245 (comment)
* adds tests that `__git_commit__` and `__version__` constants are present and that `__version__` is populated

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - Corey J. Nolet (https://github.com/cjnolet)

URL: NVIDIA#2344
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review ci improvement Improvement / enhancement to an existing function non-breaking Non-breaking change python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants