Skip to content

fix: vuln scan status type in projects_populated_ds#4084

Closed
epipav wants to merge 6 commits intomainfrom
fix/insights-projects-vulnerabilities-field-mismatch
Closed

fix: vuln scan status type in projects_populated_ds#4084
epipav wants to merge 6 commits intomainfrom
fix/insights-projects-vulnerabilities-field-mismatch

Conversation

@epipav
Copy link
Copy Markdown
Collaborator

@epipav epipav commented May 7, 2026

Note

Low Risk
No effective code changes were visible in the provided diff, so this appears to be a no-op PR with minimal risk.

Overview
No functional changes could be identified from the provided diff (it only shows +++ /dev/null with no file context or hunks). If this is unintended, re-export the full patch so reviewers can validate the intended projects_populated_ds vuln-scan status type fix.

Reviewed by Cursor Bugbot for commit 90f0c9a. Bugbot is set up for automated code reviews on this repo. Configure here.

Signed-off-by: anilb <epipav@gmail.com>
Copilot AI review requested due to automatic review settings May 7, 2026 08:26
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 7, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 3 committers have signed the CLA.

✅ themarolt
❌ epipav
❌ gaspergrom
You have signed the CLA already but the status is still pending? Let us recheck it.

skwowet and others added 4 commits May 7, 2026 10:27
…32) (#4079)

Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
…1) (#4078)

Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Uroš Marolt <uros@marolt.me>
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

⚠️ 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

Updates the Tinybird insights_projects_populated_ds datasource schema to correct the ClickHouse type of the aggregated vulnerability scan status field used by insights/analytics pipes that read from this datasource.

Changes:

  • Changed lastVulnerabilityScanStatus column type from LowCardinality(Nullable(String)) to Nullable(String) in the datasource schema.

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

`communityPlatforms` Array(String),
`communityKeywords` Array(String),
`communityLanguages` Array(String),
`status` String,
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

⚠️ 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 May 7, 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.

6 participants