Skip to content

Conversation

@AlbertoBarba
Copy link
Contributor

What this PR does / why we need it:

Checklist

  • Title of the PR starts with type and scope, (e.g. feat(agent,node-analyzer,sysdig-deploy):)
  • Chart Version bumped for the respective charts
  • Variables are documented in the README.md (or README.tpl in some charts)
  • Check GithubAction checks (like lint) to avoid merge-check stoppers
  • All test files are added in the tests folder of their respective chart and have a "_test" suffix

Copy link

@TizianoTiz987 TizianoTiz987 left a comment

Choose a reason for hiding this comment

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

lgtm

@AlbertoBarba AlbertoBarba marked this pull request as ready for review May 20, 2025 14:28
@AlbertoBarba AlbertoBarba requested a review from a team as a code owner May 20, 2025 14:28
mavimo
mavimo previously approved these changes May 20, 2025
@github-actions
Copy link
Contributor

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the stale The issue or pull request is inactive and will be closed soon label Jul 20, 2025
@francesco-furlan francesco-furlan enabled auto-merge (squash) July 21, 2025 08:14
@francesco-furlan francesco-furlan merged commit 0e2f4c7 into main Jul 21, 2025
5 checks passed
@francesco-furlan francesco-furlan deleted the docs/shield/enhrich-secure-api-token-requirements branch July 21, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale The issue or pull request is inactive and will be closed soon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants