Skip to content

chore: prevent node and pipe name clashing for vulnerability endpoints#3940

Closed
epipav wants to merge 2 commits intomainfrom
fix/vulnerability-tb-name-clash
Closed

chore: prevent node and pipe name clashing for vulnerability endpoints#3940
epipav wants to merge 2 commits intomainfrom
fix/vulnerability-tb-name-clash

Conversation

@epipav
Copy link
Copy Markdown
Collaborator

@epipav epipav commented Mar 20, 2026

Note

Low Risk
The provided diff appears empty (no file changes), so there is effectively no behavioral impact or risk beyond potential metadata/PR description changes.

Overview
No code changes are present in the provided diff (it only shows +++ /dev/null), so there are no functional modifications to review.

Written by Cursor Bugbot for commit aef4488. This will update automatically on new commits. Configure here.

Copilot AI review requested due to automatic review settings March 20, 2026 10:32
@epipav epipav self-assigned this Mar 20, 2026
@github-actions
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Renames Tinybird pipe NODEs for vulnerability endpoints to avoid name collisions between the pipe (file) name and the primary endpoint node name.

Changes:

  • Rename NODE vulnerabilities_summaryNODE vulnerabilities_summary_result
  • Rename NODE vulnerabilities_listNODE vulnerabilities_list_result
  • Rename NODE vulnerabilities_by_severityNODE vulnerabilities_by_severity_result
  • Rename NODE vulnerabilities_by_ecosystemNODE vulnerabilities_by_ecosystem_result

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
services/libs/tinybird/pipes/vulnerabilities_summary.pipe Renames the final endpoint node to *_result to avoid clashing with the pipe name.
services/libs/tinybird/pipes/vulnerabilities_list.pipe Renames the endpoint node to *_result to avoid clashing with the pipe name.
services/libs/tinybird/pipes/vulnerabilities_by_severity.pipe Renames the endpoint node to *_result to avoid clashing with the pipe name.
services/libs/tinybird/pipes/vulnerabilities_by_ecosystem.pipe Renames the endpoint node to *_result to avoid clashing with the pipe name.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link
Copy Markdown
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.

@epipav epipav closed this Mar 26, 2026
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