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

Use internal comparator instead of go-cmp #2132

Merged
merged 4 commits into from
Jul 26, 2023

Conversation

kevindiu
Copy link
Contributor

@kevindiu kevindiu commented Jul 13, 2023

Description:

Use internal comparator interface instead of directly using go-cmp library.

Related Issue:

Versions:

  • Go Version: 1.20.6
  • Docker Version: 20.10.8
  • Kubernetes Version: v1.27.3
  • NGT Version: 2.0.16

Checklist:

Special notes for your reviewer:

@sonatype-lift
Copy link

sonatype-lift bot commented Jul 13, 2023

Sonatype Lift is retiring

Sonatype Lift will be retiring on Sep 12, 2023, with its analysis stopping on Aug 12, 2023. We understand that this news may come as a disappointment, and Sonatype is committed to helping you transition off it seamlessly. If you’d like to retain your data, please export your issues from the web console.
We are extremely grateful and thank you for your support over the years.

📖 Read about the impacts and timeline

@vdaas-ci
Copy link
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 💌 /changelog - replace the PR body by changelog details
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • /rebase - rebase main
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

internal/net/dialer_test.go Show resolved Hide resolved
internal/net/dialer_test.go Show resolved Hide resolved
internal/net/dialer_test.go Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.01% 🎉

Comparison is base (f35756b) 31.14% compared to head (6a6b55a) 31.15%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2132      +/-   ##
==========================================
+ Coverage   31.14%   31.15%   +0.01%     
==========================================
  Files         339      339              
  Lines       32834    32834              
==========================================
+ Hits        10226    10230       +4     
  Misses      22155    22155              
+ Partials      453      449       -4     

see 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kevindiu kevindiu requested review from a team, kmrmt and hlts2 and removed request for a team July 13, 2023 22:27
@kevindiu kevindiu marked this pull request as ready for review July 13, 2023 22:29
@kevindiu kevindiu changed the title [Draft] Use internal comparator instead of go-cmp Use internal comparator instead of go-cmp Jul 13, 2023
@hlts2
Copy link
Contributor

hlts2 commented Jul 14, 2023

Could you please replace also this one?

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 14, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6a6b55a
Status: ✅  Deploy successful!
Preview URL: https://2ece0955.vald.pages.dev
Branch Preview URL: https://refactor-test-use-internal-g.vald.pages.dev

View logs

kmrmt
kmrmt previously approved these changes Jul 18, 2023
@ykadowak
Copy link
Contributor

Please check why DeepSource: Go is failing

@kevindiu
Copy link
Contributor Author

fix deepsource warning. thank you!

@kevindiu kevindiu force-pushed the refactor/test/use-internal-go-cmp branch from 9e2bb28 to f7e8265 Compare July 26, 2023 04:33
kevindiu and others added 4 commits July 26, 2023 14:25
Signed-off-by: kevindiu <kevin_diu@yahoo.com.hk>
Signed-off-by: kevindiu <kevin_diu@yahoo.com.hk>
Signed-off-by: kevindiu <kevin_diu@yahoo.com.hk>
@kevindiu kevindiu force-pushed the refactor/test/use-internal-go-cmp branch from f7e8265 to 6a6b55a Compare July 26, 2023 05:26
@hlts2 hlts2 merged commit bf17b93 into main Jul 26, 2023
31 checks passed
@hlts2 hlts2 deleted the refactor/test/use-internal-go-cmp branch July 26, 2023 06:28
@hlts2 hlts2 mentioned this pull request Sep 13, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants