Skip to content

Conversation

@renefloor
Copy link
Collaborator

This adds a basic stream_feeds package and workflows

For now we hope/assume we can publish this as stream_feeds, otherwise we'll rename it to stream_feed at some point.

@renefloor renefloor force-pushed the chore/initial-package-setup branch from d4d30b1 to ef345f8 Compare July 24, 2025 12:46
@codecov
Copy link

codecov bot commented Jul 24, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Co-authored-by: Sahil Kumar <xdsahil@gmail.com>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should remove this action and use the one we created in core. So any fixes can happen at one place.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah good idea, but let's take that as a separate task.

- name: 📚 Git Checkout
uses: actions/checkout@v4
- name: 📊 Verify Pana Score
uses: ./.github/actions/pana
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once removed, we can use our core action here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
uses: ./.github/actions/pana
uses: GetStream/stream-core-flutter/.github/actions/pana

Copy link
Member

@xsahil03x xsahil03x left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some suggestions, Otherwise LGTM 🚀

@renefloor renefloor merged commit c1bb1f5 into main Jul 25, 2025
5 checks passed
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.

3 participants