-
Notifications
You must be signed in to change notification settings - Fork 1
Release 0.1.1 #59
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
Release 0.1.1 #59
Conversation
feature/policy-set-parameters
Added all api functionality for organization membership and done some…
- Add WorkspaceResource model and WorkspaceResourceListOptions - Add WorkspaceResourcesService for listing workspace resources - Add workspace_resources.py example CLI tool with flag-based interface - Add comprehensive unit tests for workspace resources - Update client.py to include workspace_resources service - Update models/__init__.py with WorkspaceResource exports
- Remove urllib.parse import - Remove URL encoding for workspace_id parameter - Workspace IDs are alphanumeric with hyphens, no encoding needed - Consistent with other services in the codebase
- Remove test_list_workspace_resources_url_encoding test - URL encoding is not needed for workspace IDs - All 376 tests now passing
feat: add workspace resources functionality
contributing doc, test.md and changelog added
fix(README):Corrected the address parameter for recommended examples
docs/CONTRIBUTING.md
Outdated
|
|
||
| ## Adding New Functionality or Fixing Bugs | ||
|
|
||
| If you are adding a new endpoint, make sure to update the API coverage list where we keep track of the HCP Terraform APIs that this SDK supports. |
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.
which API coverage list ? link
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.
Each API Resource implemented should also have corresponding example added
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.
We do not have api coverage file so removing this section
Added above mention comment
iam404
left a comment
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.
@aayushsingh2502
pyproject.toml is not updated with 0.1.1 version.
Features:
CodeCleanup:
TestCases:
