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

Remove unused function #5524

Merged
merged 1 commit into from Apr 5, 2023

Conversation

fabriziomello
Copy link
Contributor

Remove unused function invalidation_threshold_htid_found.

Disable-check: force-changelog-changed

@fabriziomello fabriziomello added the tech-debt Needs refactoring and improvement tasks related to the source code and its architecture. label Apr 4, 2023
@fabriziomello fabriziomello self-assigned this Apr 4, 2023
@github-actions
Copy link

github-actions bot commented Apr 4, 2023

@RafiaSabih, @mahipv: please review this pull request.

Powered by pull-review

@codecov
Copy link

codecov bot commented Apr 4, 2023

Codecov Report

Merging #5524 (7dca1fc) into main (1fb058b) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5524      +/-   ##
==========================================
+ Coverage   90.42%   90.43%   +0.01%     
==========================================
  Files         229      229              
  Lines       53875    53851      -24     
==========================================
- Hits        48714    48700      -14     
+ Misses       5161     5151      -10     
Impacted Files Coverage Δ
tsl/src/continuous_aggs/invalidation_threshold.c 78.20% <ø> (+15.96%) ⬆️

... and 6 files with indirect coverage changes

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

Remove unused function `invalidation_threshold_htid_found`.
@fabriziomello fabriziomello enabled auto-merge (rebase) April 5, 2023 13:15
@fabriziomello fabriziomello merged commit 6440bb3 into timescale:main Apr 5, 2023
39 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Needs refactoring and improvement tasks related to the source code and its architecture.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants