Skip to content

ci: use plain version tags so Read the Docs can version releases (DM-3216) - #2764

Merged
yangru96 merged 1 commit into
masterfrom
ci/rtd-native-versioning-dm-3216
Aug 12, 2026
Merged

ci: use plain version tags so Read the Docs can version releases (DM-3216)#2764
yangru96 merged 1 commit into
masterfrom
ci/rtd-native-versioning-dm-3216

Conversation

@yangru96

Copy link
Copy Markdown
Contributor

Description

Set "include-component-in-tag": false so release tags become v8.13.0 instead of cognite-sdk-python-v8.13.0.

Read the Docs cannot parse the current tag format as a version, so no release can be activated as a docs version. The project only has latest (built from master), meaning users always read docs for unreleased code. With plain tags we can activate release versions on RTD and make stable the default.

Notes:

Follow-ups: activate versions and set default to stable on RTD (needs admin access), then update the README badge/links that are hardcoded to /en/latest/.

Checklist:

  • Tests added/updated.
  • Documentation updated. Documentation is generated from docstrings - these must be updated according to your change.
    If a new method has been added it should be referenced in cognite.rst in order to generate docs based on its docstring.
  • The PR title follows the Conventional Commit spec.

@yangru96
yangru96 requested review from a team as code owners August 12, 2026 12:52

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request updates the release-please configuration by setting 'include-component-in-tag' to false. There are no review comments, and I have no feedback to provide.

@haakonvt haakonvt left a comment

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.

LGTM

@haakonvt haakonvt added the waiting-for-risk-review Waiting for a member of the risk review team to take an action label Aug 12, 2026
@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.77%. Comparing base (bef4f27) to head (6e945ec).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2764      +/-   ##
==========================================
- Coverage   93.83%   93.77%   -0.06%     
==========================================
  Files         513      513              
  Lines       52620    52620              
==========================================
- Hits        49377    49346      -31     
- Misses       3243     3274      +31     

see 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yangru96
yangru96 enabled auto-merge August 12, 2026 13:01
@andersfylling

Copy link
Copy Markdown
Contributor

🦄

@andersfylling andersfylling added risk-review-ongoing Risk review is in progress waiting-for-team Waiting for the submitter or reviewer of the PR to take an action and removed waiting-for-risk-review Waiting for a member of the risk review team to take an action labels Aug 12, 2026
@andersfylling andersfylling self-assigned this Aug 12, 2026
@yangru96
yangru96 added this pull request to the merge queue Aug 12, 2026
Merged via the queue into master with commit 2ea6d56 Aug 12, 2026
22 checks passed
@yangru96
yangru96 deleted the ci/rtd-native-versioning-dm-3216 branch August 12, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk-review-ongoing Risk review is in progress waiting-for-team Waiting for the submitter or reviewer of the PR to take an action

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants