Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: remove old Python packages #6985

Merged
merged 1 commit into from
May 25, 2023
Merged

Conversation

dzhu
Copy link
Contributor

@dzhu dzhu commented May 25, 2023

Description

The determined-cli, determined-common, and determined-deploy packages, corresponding to the cli, common, and deploy directories in this repo, were deprecated over two years ago, so it is high time to remove them entirely.

Test Plan

  • look through all instances of the strings "cli", "common", and "deploy" in the repo
  • CI

Commentary

I guess this is a breaking change, though the hope is that no one will be affected by it.

@dzhu dzhu requested a review from a team as a code owner May 25, 2023 22:28
@dzhu dzhu requested a review from a team as a code owner May 25, 2023 22:28
@dzhu dzhu requested a review from loksonarius May 25, 2023 22:28
@cla-bot cla-bot bot added the cla-signed label May 25, 2023
@determined-ci determined-ci added the documentation Improvements or additions to documentation label May 25, 2023
The `determined-cli`, `determined-common`, and `determined-deploy`
packages, corresponding to the `cli`, `common`, and `deploy` directories
in this repo, were deprecated over two years ago, so it is high time to
remove them entirely.
@dzhu dzhu merged commit daab280 into determined-ai:main May 25, 2023
@dzhu dzhu deleted the remove-old-python branch May 25, 2023 23:41
erikwilson pushed a commit that referenced this pull request Jun 1, 2023
The `determined-cli`, `determined-common`, and `determined-deploy`
packages, corresponding to the `cli`, `common`, and `deploy` directories
in this repo, were deprecated over two years ago, so it is high time to
remove them entirely.
wes-turner pushed a commit that referenced this pull request Jun 5, 2023
The `determined-cli`, `determined-common`, and `determined-deploy`
packages, corresponding to the `cli`, `common`, and `deploy` directories
in this repo, were deprecated over two years ago, so it is high time to
remove them entirely.
wes-turner pushed a commit that referenced this pull request Jun 5, 2023
The `determined-cli`, `determined-common`, and `determined-deploy`
packages, corresponding to the `cli`, `common`, and `deploy` directories
in this repo, were deprecated over two years ago, so it is high time to
remove them entirely.
@dannysauer dannysauer added this to the 0.23.0 milestone Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants