yazdipour / OmnivoreQL

@Omnivore-app API client for Python
https://pypi.org/project/omnivoreql/
MIT License
69 stars 6 forks source link

Bugfix/better set_label name #24

Closed yazdipour closed 5 months ago

yazdipour commented 5 months ago

ℹ Additional Information

Summary by Sourcery

This pull request refactors the query loading mechanism, introduces new methods for setting page labels, and renames a parameter for consistency. It also includes updates and additions to the test suite to cover these changes.

sourcery-ai[bot] commented 5 months ago

Reviewer's Guide by Sourcery

This pull request refactors the query loading mechanism in omnivoreql.py to load queries on demand using a new _get_query method. It also renames clientRequestId to client_request_id for consistency and adds new methods for setting labels on pages. Corresponding tests in tests/test_omnivoreql.py have been updated and new tests added to cover the new functionality.

File-Level Changes

Files Changes
omnivoreql/omnivoreql.py
tests/test_omnivoreql.py
Refactored query loading mechanism and added new methods for setting page labels. Updated tests to cover new methods and refactored existing tests for consistency.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - You can change your review settings at any time by accessing your [dashboard](https://sourcery.ai/dashboard): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.