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

Fixed Coverity Scan Warnings #5672

Merged
merged 1 commit into from
May 10, 2023
Merged

Conversation

shhnwz
Copy link
Contributor

@shhnwz shhnwz commented May 10, 2023

Unused Value reported during coverity scan
link: https://scan4.scan.coverity.com/reports.htm#v56957/p12995

@github-actions
Copy link

@mkindahl, @konskov: please review this pull request.

Powered by pull-review

@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Merging #5672 (67284f2) into main (8e69a99) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5672      +/-   ##
==========================================
- Coverage   90.98%   90.96%   -0.03%     
==========================================
  Files         230      230              
  Lines       54473    54465       -8     
==========================================
- Hits        49564    49545      -19     
- Misses       4909     4920      +11     
Impacted Files Coverage Δ
src/bgw/job.c 93.15% <ø> (-0.34%) ⬇️
tsl/src/nodes/decompress_chunk/decompress_chunk.c 94.51% <ø> (-0.01%) ⬇️

... and 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@shhnwz shhnwz linked an issue May 10, 2023 that may be closed by this pull request
@shhnwz shhnwz merged commit bd36afe into timescale:main May 10, 2023
100 of 102 checks passed
@shhnwz shhnwz deleted the 5669-coverity-warnings branch May 10, 2023 08:54
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.

[Bug]:Fix Coverity Warnings
4 participants