Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: make verify permission error #7003

Merged

Conversation

mateusoliveira43
Copy link
Contributor

@mateusoliveira43 mateusoliveira43 commented Oct 23, 2023

Please add a summary of your change

Fix make verify command, which is silently failing with Permission denied error.

Examples:

Does your change fix a particular issue?

No.

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

@mateusoliveira43
Copy link
Contributor Author

Do you want me to run make update in this PR?

@shubham-pampattiwar
Copy link
Collaborator

yes, that would be ideal, Thank you @mateusoliveira43

@mateusoliveira43
Copy link
Contributor Author

/kind changelog-not-required

@github-actions github-actions bot added the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Oct 23, 2023
Signed-off-by: Mateus Oliveira <msouzaol@redhat.com>
Signed-off-by: Mateus Oliveira <msouzaol@redhat.com>
@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #7003 (3bc23ae) into main (30bf6bd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #7003   +/-   ##
=======================================
  Coverage   61.05%   61.05%           
=======================================
  Files         252      252           
  Lines       26855    26855           
=======================================
  Hits        16395    16395           
  Misses       9305     9305           
  Partials     1155     1155           

@qiuming-best qiuming-best merged commit 1fcdc20 into vmware-tanzu:main Oct 30, 2023
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants