8000 PG-1464 CodeChecker integration by artemgavrilov · Pull Request #339 · percona/postgres · GitHub
[go: up one dir, main page]
Skip to content

PG-1464 CodeChecker integration #339

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 4, 2025
Merged

PG-1464 CodeChecker integration #339

merged 1 commit into from
Jun 4, 2025

Conversation

artemgavrilov
Copy link
Collaborator
@artemgavrilov artemgavrilov commented May 15, 2025 •
edited
Loading

https://perconadev.atlassian.net/browse/PG-1464

This PR adds workflow with CodeChecker run. Underhood it runs clang-tidy static analyzer. Results published to CodeChecker server. UI available here: https://codechecker.percona.com/

Server will store only latest results for branch/PR.

@artemgavrilov artemgavrilov force-pushed the PG-1464-static-analyzer branch from 98eb73f to d077be8 Compare June 2, 2025 13:56
@codecov-commenter
Copy link
codecov-commenter commented Jun 2, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.39%. Comparing base (225a23c) to head (09c223e).

❌ Your project status has failed because the head coverage (85.39%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files 
@@                Coverage Diff                 @@
##           TDE_REL_17_STABLE     #339   +/-   ##
==================================================
  Coverage              85.39%   85.39%           
==================================================
  Files                     22       22           
  Lines                   2602     2602           
  Branches                 393      393           
==================================================
  Hits                    2222     2222           
  Misses                   304      304           
  Partials                  76       76
Components Coverage Δ
access 84.20% <ø> (ø)
catalog 88.20% <ø> (ø)
common 91.80% <ø> (ø)
encryption 73.45% <ø> (ø)
keyring 72.00% <ø> (ø)
src 91.40% <ø> (ø)
smgr 97.41% <ø> (ø)
transam ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@artemgavrilov artemgavrilov force-pushed the PG-1464-static-analyzer branch 2 times, most recently from 1ef980e to 4b417fe Compare June 2, 2025 22:43
@artemgavrilov artemgavrilov changed the title PG-1464 Static Analyzer PG-1464 CodeChecker integration Jun 2, 2025
@artemgavrilov
PG-1464 Add CodeChecker integration
09c223e 
Add CI workflow for CodeChecker. It runs clang-tidy static analyzer and
stores results on CodeChecker server.
@artemgavrilov artemgavrilov force-pushed the PG-1464-static-analyzer branch from a1a8a06 to 09c223e Compare June 2, 2025 23:43
@artemgavrilov artemgavrilov marked this pull request as ready for review June 3, 2025 07:12
@artemgavrilov artemgavrilov merged commit 04e09f2 into TDE_REL_17_STABLE Jun 4, 2025
17 checks passed
@artemgavrilov artemgavrilov deleted the PG-1464-static-analyzer branch June 4, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeltz jeltz jeltz approved these changes

@dAdAbird dAdAbird dAdAbird approved these changes

@dutow dutow Awaiting requested review from dutow

@AndersAstrand AndersAstrand Awaiting requested review from AndersAstrand

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@artemgavrilov @codecov-commenter @jeltz @dAdAbird
0