Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cb38875e08
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | `roboflow import <folder> -p <project>` | `roboflow import <folder> -p <project>` | | ||
| | `roboflow download <url> -f <format>` | `roboflow download <url> -f <format>` | | ||
| | `roboflow infer <file> -m <model>` | `roboflow infer <file> -m <model>` | | ||
| | `roboflow config show` | `roboflow whoami` | |
There was a problem hiding this comment.
config show to a supported Python CLI command
The migration table maps roboflow config show to roboflow whoami, but whoami is not part of the documented Python CLI subcommands, so users migrating scripts will hit an unknown-command error instead of getting equivalent behavior. Please replace this with a real supported command (or remove the mapping) so scripted migrations do not break.
Useful? React with 👍 / 👎.
2 participants
What does this PR do?
Deprecates the Node.js CLI in favor of the Roboflow Python package. Replaces the README with a deprecation notice, a feature comparison table showing the Python package is a superset, and a command mapping to help users migrate their scripts.
Type of Change
Testing
Test details:
Verified README renders correctly with the deprecation notice, comparison table, and command mapping.
Checklist