Skip to content

Conversation

chgl
Copy link
Contributor

@chgl chgl commented Nov 11, 2023

Description

Closes #555 . Alternatively, it could also be an option if required.

Type of Change

[ ] Bug Fix
[ ] New Feature
[ ] Breaking Change
[x] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@chgl chgl requested a review from a team as a code owner November 11, 2023 14:42
@chgl chgl changed the title fex(helm): remove helm.sh/hook annotations fix(helm): remove helm.sh/hook annotations Nov 13, 2023
@FrimIdan FrimIdan enabled auto-merge December 20, 2023 06:13
@FrimIdan FrimIdan added this pull request to the merge queue Dec 20, 2023
Merged via the queue into openclarity:main with commit 6ef30a4 Dec 20, 2023
@chgl chgl deleted the remove-helm-hooks branch August 10, 2024 16:05
ramizpolic pushed a commit that referenced this pull request Aug 10, 2024
ramizpolic pushed a commit that referenced this pull request Aug 11, 2024
ramizpolic pushed a commit that referenced this pull request Aug 12, 2024
ramizpolic pushed a commit that referenced this pull request Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Consider removing or making the helm.sh/hook: pre-install annotations optional
2 participants