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

code clean for repository #6768

Merged
merged 1 commit into from
Sep 12, 2023
Merged

Conversation

yanggangtony
Copy link
Contributor

Thank you for contributing to Velero!

Please add a summary of your change

Reused the const parm.

Does your change fix a particular issue?

Fixes #(issue)

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.

Signed-off-by: yanggang <[email protected]>
@yanggangtony
Copy link
Contributor Author

/kind changelog-not-required

@github-actions github-actions bot requested review from sseago and ywk253100 September 5, 2023 14:38
@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 Sep 5, 2023
@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Merging #6768 (92c1c9e) into main (c9e1ade) will not change coverage.
The diff coverage is 75.00%.

@@           Coverage Diff           @@
##             main    #6768   +/-   ##
=======================================
  Coverage   60.38%   60.38%           
=======================================
  Files         242      242           
  Lines       25982    25982           
=======================================
  Hits        15689    15689           
  Misses       9189     9189           
  Partials     1104     1104           
Files Changed Coverage Δ
pkg/repository/config/gcp.go 58.33% <0.00%> (ø)
pkg/repository/backup_repo_op.go 100.00% <100.00%> (ø)
pkg/repository/config/aws.go 39.34% <100.00%> (ø)

@ywk253100 ywk253100 merged commit ec11a5a into vmware-tanzu:main Sep 12, 2023
@yanggangtony yanggangtony deleted the small-clean branch September 12, 2023 10:41
reasonerjt pushed a commit to reasonerjt/velero that referenced this pull request Sep 28, 2023
reasonerjt pushed a commit to reasonerjt/velero that referenced this pull request Sep 28, 2023
ywk253100 added a commit that referenced this pull request Sep 28, 2023
[Cherry-pick v1.12] code clean for repository (#6768)
kaovilai pushed a commit to kaovilai/velero that referenced this pull request Oct 2, 2023
kaovilai pushed a commit to kaovilai/velero that referenced this pull request Oct 2, 2023
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 target/1.12.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants