Skip to content
This repository has been archived by the owner on Dec 23, 2024. It is now read-only.

resolve issue #38 #45

Merged
merged 1 commit into from
Sep 3, 2024
Merged

resolve issue #38 #45

merged 1 commit into from
Sep 3, 2024

Conversation

nurzhan-saktaganov
Copy link
Collaborator

  • simplify DiscoveryAllBuckets
  • get rid of a suspicious if

@nurzhan-saktaganov nurzhan-saktaganov force-pushed the nsaktaganov_resolve_issue_38 branch from faa5bf3 to fd1ae50 Compare August 22, 2024 14:30
@nurzhan-saktaganov nurzhan-saktaganov changed the title resolve issue 38 WIP: resolve issue #38 Aug 22, 2024
@nurzhan-saktaganov nurzhan-saktaganov force-pushed the nsaktaganov_resolve_issue_39 branch from 9a80119 to 3ca9b83 Compare August 23, 2024 16:12
@nurzhan-saktaganov nurzhan-saktaganov force-pushed the nsaktaganov_resolve_issue_38 branch from fd1ae50 to 989d44a Compare August 26, 2024 10:32
discovery.go Outdated Show resolved Hide resolved
nurzhan-saktaganov added a commit that referenced this pull request Aug 26, 2024
@nurzhan-saktaganov nurzhan-saktaganov linked an issue Aug 27, 2024 that may be closed by this pull request
@nurzhan-saktaganov nurzhan-saktaganov added invalid This doesn't seem right performance and removed performance labels Sep 1, 2024
@nurzhan-saktaganov nurzhan-saktaganov force-pushed the nsaktaganov_resolve_issue_39 branch from 3ca9b83 to c5012fb Compare September 2, 2024 13:22
@nurzhan-saktaganov nurzhan-saktaganov force-pushed the nsaktaganov_resolve_issue_38 branch from 28c9833 to b0ba924 Compare September 3, 2024 09:22
@nurzhan-saktaganov nurzhan-saktaganov changed the base branch from nsaktaganov_resolve_issue_39 to master September 3, 2024 09:23
@codecov-commenter
Copy link

codecov-commenter commented Sep 3, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 30.74%. Comparing base (dfa83c2) to head (37cc7ee).

Files with missing lines Patch % Lines
discovery.go 0.00% 13 Missing ⚠️
api.go 0.00% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
+ Coverage   30.31%   30.74%   +0.42%     
==========================================
  Files          10       10              
  Lines         719      709      -10     
==========================================
  Hits          218      218              
+ Misses        487      477      -10     
  Partials       14       14              

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

* simplify DiscoveryAllBuckets: remove a suspicious if
* fix warning from linter v1.60.3
@nurzhan-saktaganov nurzhan-saktaganov force-pushed the nsaktaganov_resolve_issue_38 branch from 62fed9b to 37cc7ee Compare September 3, 2024 16:45
@nurzhan-saktaganov nurzhan-saktaganov changed the title WIP: resolve issue #38 resolve issue #38 Sep 3, 2024
@nurzhan-saktaganov nurzhan-saktaganov marked this pull request as ready for review September 3, 2024 16:46
@KaymeKaydex KaymeKaydex merged commit 5f5cc35 into master Sep 3, 2024
2 checks passed
@nurzhan-saktaganov nurzhan-saktaganov deleted the nsaktaganov_resolve_issue_38 branch September 17, 2024 16:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A suspicious condition in 'if' operator
3 participants