Skip to content

add feed support, tests and integration tests#57

Merged
mihirvala08 merged 10 commits into
google:mainfrom
xmikew:feature/feeds
Jul 22, 2025
Merged

add feed support, tests and integration tests#57
mihirvala08 merged 10 commits into
google:mainfrom
xmikew:feature/feeds

Conversation

@xmikew
Copy link
Copy Markdown
Contributor

@xmikew xmikew commented Jul 1, 2025

No description provided.

@google-cla
Copy link
Copy Markdown

google-cla Bot commented Jul 1, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@xmikew
Copy link
Copy Markdown
Contributor Author

xmikew commented Jul 1, 2025

Fixes #30

@xmikew
Copy link
Copy Markdown
Contributor Author

xmikew commented Jul 1, 2025

Forgot to add the google-style docstrings, will fix up when I get a chance.

@mihirvala08
Copy link
Copy Markdown
Collaborator

Could you provide example usage as well? It will help for updating documentation and adding missing tests for CLI.

Thanks.

@mihirvala08
Copy link
Copy Markdown
Collaborator

Could you provide example usage as well? It will help for updating documentation and adding missing tests for CLI.

Thanks.

@xmikew I've added example and missing tests along with updated documentation.

@mihirvala08 mihirvala08 self-requested a review July 22, 2025 05:29
@mihirvala08 mihirvala08 merged commit d4b0bc5 into google:main Jul 22, 2025
6 checks passed
@xmikew
Copy link
Copy Markdown
Contributor Author

xmikew commented Jul 22, 2025

Thank you @mihirvala-crestdata, much appreciated on the assist and cleanup.

@xmikew xmikew deleted the feature/feeds branch September 11, 2025 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants