Skip to content

Comments

Add Peer Review verification action#41

Open
niccrane wants to merge 5 commits intoKoenkk:mainfrom
niccrane:helm_docs_ci
Open

Add Peer Review verification action#41
niccrane wants to merge 5 commits intoKoenkk:mainfrom
niccrane:helm_docs_ci

Conversation

@niccrane
Copy link
Contributor

@niccrane niccrane commented Mar 17, 2025

Working on addressing #39

  • Lints the chart
  • Fails on helm-docs diff
  • Post PR comment for clarity

@niccrane
Copy link
Contributor Author

I think the PR comment will only work if the PR is not coming from a fork.

@Koenkk Koenkk requested a review from jlpedrosa March 17, 2025 19:48
@jlpedrosa
Copy link
Collaborator

I think it is actually doing what it should? But generate needs to be run again? are we actually using the same tool? I don't mind which one as long as they are not fighting each other.

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.

2 participants