-
Notifications
You must be signed in to change notification settings - Fork 726
fix: use repos_to_channel for PR & issues #3810
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
2 similar comments
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
This pull request updates the way channels are filtered in two Tinybird pipes to use the
repos_to_channelsmapping instead of directly querying therepositoriestable. This change ensures that the filtering logic is consistent with the current mapping between repositories and channels.Query logic improvements:
issues_average_resolve_velocity.pipeto select channels from therepos_to_channelstable instead of therepositoriestable.pull_requests_average_resolve_velocity.pipeto select channels from therepos_to_channelstable instead of therepositoriestable.Note
Low Risk
Low risk SQL-only changes to channel filtering; main impact is which repos are included by default when
reposisn’t provided.Overview
Aligns Tinybird pipes to consistently filter activity by
repos_to_channelsrather than directly readingrepositoriesURLs, fixing default/fallback repo scoping for issue and PR resolve-velocity metrics.Simplifies
pull_requests_filtered.pipeto always constrainpra.channelviarepos_to_channels(removing the prior conditional fallback), and updatesrepos_to_channels.pipeso thereposparameter is optional and defaults to all non-excluded, enabled (and non-deleted) repositories.Written by Cursor Bugbot for commit 4a8a3a1. This will update automatically on new commits. Configure here.