Skip to content

fix: add type checking to message sanitation#755

Merged
merll merged 6 commits into
mainfrom
APL-916
Jun 25, 2025
Merged

fix: add type checking to message sanitation#755
merll merged 6 commits into
mainfrom
APL-916

Conversation

@merll
Copy link
Copy Markdown
Contributor

@merll merll commented Jun 23, 2025

This PR adds a fix and regression test to a utility function that removes Git credentials from log output.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 23, 2025

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements
50.52% (+0.05% 🔼)
2277/4507
🔴 Branches
35.22% (+0.19% 🔼)
368/1045
🔴 Functions
42.55% (+0.16% 🔼)
417/980
🔴 Lines
50.9% (+0.06% 🔼)
2140/4204
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🔴 git.ts
15.22% (-0.05% 🔻)
6.58% 7.32%
15.87% (-0.06% 🔻)

Test suite run success

332 tests passing in 12 suites.

Report generated by 🧪jest coverage report action from 32bfb90

Comment thread src/utils.ts Outdated
Copy link
Copy Markdown
Collaborator

@ferruhcihan ferruhcihan left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread src/utils.ts Outdated
@merll merll merged commit 242b743 into main Jun 25, 2025
8 checks passed
@merll merll deleted the APL-916 branch June 25, 2025 09:55
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.

4 participants