Skip to content

Conversation

@nathanroberts55
Copy link
Contributor

Updated to keep APIs under single TTXClient

Also using config.yml

Updated generate script

@nathanroberts55 nathanroberts55 requested a review from dylhack May 6, 2025 22:10
@nathanroberts55 nathanroberts55 added documentation Improvements or additions to documentation enhancement New feature or request labels May 6, 2025
@nathanroberts55 nathanroberts55 removed the request for review from dylhack May 6, 2025 22:10
@dylhack dylhack requested a review from Copilot May 6, 2025 22:13
@dylhack dylhack self-requested a review May 6, 2025 22:14
Copy link

Copilot AI left a comment

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 consolidates multiple API endpoints under a single TTXClientApi, updates configuration settings, and revises documentation and setup details to reflect the new API structure.

  • Removed imports of individual APIs in favor of TTXClientApi.
  • Added a new configuration file (config.yml) for API generation.
  • Updated tests and documentation to use TTXClientApi and the new repository URL.

Reviewed Changes

Copilot reviewed 9 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
ttx/api/init.py Replaced multiple API imports with a single TTXClientApi import.
ttx/init.py Updated API imports to use TTXClientApi.
test/test_ttx_client_api.py Introduced test stubs for the new TTXClientApi endpoints.
setup.py Updated the repository URL for the package.
pyproject.toml Minor text change in description.
config.yml Added configuration for API generation and tagging operations.
README.md Revised installation, usage examples, and API endpoint documentation.
Files not reviewed (3)
  • .openapi-generator/FILES: Language not supported
  • generate.ps1: Language not supported
  • git_push.sh: Language not supported
Comments suppressed due to low confidence (1)

test/test_ttx_client_api.py:27

  • The test case stubs currently use 'pass' and do not verify the API functionality. Consider adding assertions and concrete test scenarios to ensure proper test coverage for TTXClientApi.
pass

@dylhack dylhack merged commit 224245c into main May 6, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants