Skip to content

Fix: remove dataset link#368

Open
l-mansouri wants to merge 13 commits into
mainfrom
rm-dataset-link
Open

Fix: remove dataset link#368
l-mansouri wants to merge 13 commits into
mainfrom
rm-dataset-link

Conversation

@l-mansouri

@l-mansouri l-mansouri commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Overview

This PR removes the datasets link command and removes support for the v1 linking endpoint that got deprecated.

JIRA

Please add here as many related tasks this PR covers with its brief description, if more than one ticket

LP-112583

Changes

Removes cloudos datasets link command and support for a deprecated endpoint

Acceptance Criteria

Please add here as many scenarios as in the Story

Normally this acceptance criteria is tested in ADAPT workspace in PROD

Scenario 1: Command is no longer available Screenshot 2026-06-30 at 15 42 33
Scenario 2: User is guided to the supported alternatives Screenshot 2026-06-30 at 16 14 50
Scenario 3: Other datasets subcommands remain functional Screenshot 2026-06-30 at 16 40 13
Scenario 4: Help text and docs are clean Screenshot 2026-06-30 at 15 42 33

Copilot AI review requested due to automatic review settings June 30, 2026 14:42
@l-mansouri l-mansouri marked this pull request as draft June 30, 2026 14:45
@l-mansouri l-mansouri marked this pull request as ready for review June 30, 2026 14:45

Copilot AI 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.

Pull request overview

This PR removes the deprecated cloudos datasets link CLI subcommand (and its documentation) and bumps the CLI patch version accordingly, steering users toward cloudos interactive-session link / --link alternatives.

Changes:

  • Removed the cloudos datasets link command implementation from the datasets CLI module.
  • Updated README to remove datasets link documentation and point users to the supported linking commands.
  • Bumped version to 2.94.2 and added release notes to the changelog.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

File Description
README.md Removes datasets link documentation and replaces it with guidance to interactive-session link / --link.
cloudos_cli/datasets/cli.py Deletes the datasets link command implementation (and related import).
cloudos_cli/_version.py Bumps package version to 2.94.2.
CHANGELOG.md Adds a v2.94.2 entry describing removal of datasets link.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CHANGELOG.md

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comment thread cloudos_cli/datasets/cli.py
Comment thread tests/test_datasets/test_link_files.py
Comment thread CHANGELOG.md
@l-mansouri l-mansouri marked this pull request as draft June 30, 2026 15:15
@l-mansouri l-mansouri marked this pull request as ready for review June 30, 2026 15:15
@l-mansouri l-mansouri requested a review from Copilot June 30, 2026 15:29

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Comment thread cloudos_cli/interactive_session/link.py Outdated
Comment thread cloudos_cli/interactive_session/link.py Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Comment thread cloudos_cli/interactive_session/link.py Outdated
Comment thread README.md Outdated

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Comment thread cloudos_cli/interactive_session/link.py Outdated
Comment thread cloudos_cli/datasets/cli.py Outdated

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Comment thread tests/test_datasets/test_link.py
Comment thread cloudos_cli/interactive_session/link.py Outdated

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comment thread cloudos_cli/datasets/cli.py
Comment thread cloudos_cli/interactive_session/link.py Outdated
Comment thread README.md Outdated
l-mansouri and others added 3 commits July 1, 2026 09:12
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

@danielboloc danielboloc left a comment

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.

LGTM

Only some doubts:

  1. When I do cloudos datasets link --help shouldn't we also add the deprecation error?
  2. I would remove the Usage: cloudos datasets link [OPTIONS] if possible from click
Image

@l-mansouri

Copy link
Copy Markdown
Collaborator Author

@danielboloc thanks for the comment! I have updated the behaviour
Screenshot 2026-07-02 at 10 28 18

@l-mansouri l-mansouri marked this pull request as draft July 2, 2026 08:29
@l-mansouri l-mansouri marked this pull request as ready for review July 2, 2026 08:29
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.

3 participants