Skip to content

chore(keeper): add license and security check#34454

Merged
zitsen merged 3 commits intomainfrom
fix/main/keeper/sec
Feb 2, 2026
Merged

chore(keeper): add license and security check#34454
zitsen merged 3 commits intomainfrom
fix/main/keeper/sec

Conversation

@qevolg
Copy link
Copy Markdown
Contributor

@qevolg qevolg commented Jan 30, 2026

Description

chore: add license and security check

Issue(s)

Checklist

Please check the items in the checklist if applicable.

  • Is the user manual updated?
  • Are the test cases passed and automated?
  • Is there no significant decrease in test coverage?

Copilot AI review requested due to automatic review settings January 30, 2026 03:41
@qevolg qevolg requested a review from feici02 as a code owner January 30, 2026 03:41
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

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

This PR updates the taosKeeper security/audit workflow to both broaden its trigger scope and add license checking alongside vulnerability scanning.

Changes:

  • Renames the workflow from “taosKeeper govulncheck” to “taosKeeper Audit” and normalizes quoting in the cron/schedule configuration.
  • Expands pull_request and push path filters from tools/keeper/go.mod to tools/keeper/**, ensuring the audit runs for any keeper-related change.
  • Refactors the audit job steps to: install and run govulncheck from tools/keeper, install go-licenses, and run go-licenses check ./... from tools/keeper as an additional license-compliance check.

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

@qevolg qevolg force-pushed the fix/main/keeper/sec branch from 6523e8b to cc843dc Compare January 30, 2026 05:40
@qevolg qevolg force-pushed the fix/main/keeper/sec branch from cc843dc to 74d90c3 Compare January 30, 2026 05:44
@qevolg qevolg force-pushed the fix/main/keeper/sec branch from 37fafba to e6512b8 Compare January 30, 2026 05:55
@qevolg qevolg changed the title Fix/main/keeper/sec chore: add license and security check Jan 30, 2026
@qevolg qevolg changed the title chore: add license and security check chore(keeper): add license and security check Jan 30, 2026
@zitsen zitsen merged commit 652f232 into main Feb 2, 2026
11 of 12 checks passed
@zitsen zitsen deleted the fix/main/keeper/sec branch February 2, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants