Skip to content

Conversation

@mbani01
Copy link
Contributor

@mbani01 mbani01 commented Jan 30, 2026

This pull request updates the way channels are filtered in two Tinybird pipes to use the repos_to_channels mapping instead of directly querying the repositories table. This change ensures that the filtering logic is consistent with the current mapping between repositories and channels.

Query logic improvements:

  • Updated the fallback channel filtering in issues_average_resolve_velocity.pipe to select channels from the repos_to_channels table instead of the repositories table.
  • Updated the fallback channel filtering in pull_requests_average_resolve_velocity.pipe to select channels from the repos_to_channels table instead of the repositories table.

Note

Low Risk
Low risk SQL-only changes to channel filtering; main impact is which repos are included by default when repos isn’t provided.

Overview
Aligns Tinybird pipes to consistently filter activity by repos_to_channels rather than directly reading repositories URLs, fixing default/fallback repo scoping for issue and PR resolve-velocity metrics.

Simplifies pull_requests_filtered.pipe to always constrain pra.channel via repos_to_channels (removing the prior conditional fallback), and updates repos_to_channels.pipe so the repos parameter 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.

@mbani01 mbani01 requested a review from gaspergrom January 30, 2026 14:49
@mbani01 mbani01 self-assigned this Jan 30, 2026
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

2 similar comments
@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

Copy link

@cursor cursor bot left a 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.

@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions
Copy link
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@mbani01 mbani01 merged commit be0e659 into main Jan 30, 2026
16 checks passed
@mbani01 mbani01 deleted the fix/gerrit_widgets_missing_data branch January 30, 2026 15:20
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