-
Notifications
You must be signed in to change notification settings - Fork 20
Entity Management Script #245
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
base: main
Are you sure you want to change the base?
Conversation
| 9. list-all | ||
| - List all entities (products and projects) with their tags and labels. | ||
|
|
||
| All deletion commands prompt for confirmation before execution unless dry-run is used. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would default dryRun to true instead of prompting for confirmation. We would want to encourage the ability for this to be ran in a pipeline. Requiring input makes it unusable from that standpoint.
| load_dotenv() | ||
|
|
||
| ORG_UUID = os.getenv("ORG_UUID") | ||
| JWT_TOKEN = os.getenv("JWT_TOKEN") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The script itself should handle the auth instead of relying on an Environment variable. The token can expire and generally not something you want to store.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to create a ReadMe on how to use it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- - - [ ]
No description provided.