yazdipour / OmnivoreQL

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

Bugfix/updated schema for search-get_article #31

Closed yazdipour closed 4 months ago

yazdipour commented 4 months ago

Closes #29

Summary by Sourcery

This pull request updates the GraphQL schema to use a new fragment for search results, adds an optional parameter to fetch article content, fixes a minor README formatting issue, and enhances the test coverage for article fetching.

sourcery-ai[bot] commented 4 months ago

Reviewer's Guide by Sourcery

This pull request addresses issue #29 by updating the GraphQL schema to use a fragment for the Search query, improving reusability and maintainability. It also adds support for including content in the get_article method and updates the corresponding tests to verify this functionality. Additionally, a minor fix was made to the README.md file to correct a duplicated header.

File-Level Changes

Files Changes
omnivoreql/queries/Search.graphql
omnivoreql/omnivoreql.py
Updated the Search query to use a fragment for better reusability and added support for including content in the get_article method.
tests/test_omnivoreql.py
omnivoreql/omnivoreql.py
Enhanced the test_get_articles test to verify the inclusion of content and updated the get_article method to support the new include_content parameter.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - 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.