Skip to content

Conversation

Naeem-Akhter
Copy link
Collaborator

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.20%. Comparing base (f89f408) to head (4062337).
⚠️ Report is 7 commits behind head on TDE_REL_17_STABLE.

❌ Your project status has failed because the head coverage (84.20%) 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     #585   +/-   ##
==================================================
  Coverage              84.20%   84.20%           
==================================================
  Files                     25       25           
  Lines                   3247     3247           
  Branches                 511      511           
==================================================
  Hits                    2734     2734           
  Misses                   400      400           
  Partials                 113      113           
Components Coverage Δ
access 84.72% <ø> (ø)
catalog 87.65% <ø> (ø)
common 77.77% <ø> (ø)
encryption 72.97% <ø> (ø)
keyring 73.21% <ø> (ø)
src 94.18% <ø> (ø)
smgr 96.53% <ø> (ø)
transam ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@AndersAstrand AndersAstrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a body to the commit message that explains what problem this solves.

Also the commit message header should be in imperative mood without an ending period

Ie, it should start with "Add" rather than "Adding" 🙂

Some of the files in src/test/perl/PostgreSQL/Test directory that are used in
the tap tests are not being bundled in RHEL's devel & Debian's client
packages. This commit ensures that all files present in the mentioned directory
are present in the devel and client packages.
@Naeem-Akhter
Copy link
Collaborator Author

Please add a body to the commit message that explains what problem this solves.

Also the commit message header should be in imperative mood without an ending period

Ie, it should start with "Add" rather than "Adding" 🙂

Updated as per feedback.

@Naeem-Akhter Naeem-Akhter merged commit c7c9b73 into TDE_REL_17_STABLE Sep 23, 2025
19 checks passed
@Naeem-Akhter Naeem-Akhter deleted the devel-package-fix branch September 23, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants