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 normalizer processor #400

Merged
merged 11 commits into from
Jun 24, 2024
Merged

remove normalizer processor #400

merged 11 commits into from
Jun 24, 2024

Conversation

dtrai2
Copy link
Contributor

@dtrai2 dtrai2 commented Jun 5, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jun 5, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.10 ⚠️

Comparison is base (466a809) 92.21% compared to head (95a1195) 92.11%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #400      +/-   ##
==========================================
- Coverage   92.21%   92.11%   -0.10%     
==========================================
  Files         133      131       -2     
  Lines        9452     9086     -366     
==========================================
- Hits         8716     8370     -346     
+ Misses        736      716      -20     
Impacted Files Coverage Δ
logprep/connector/file/input.py 100.00% <ø> (+0.56%) ⬆️
logprep/registry.py 100.00% <ø> (ø)

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dtrai2 dtrai2 requested review from ekneg54 and ppcad June 5, 2023 12:51
@dtrai2 dtrai2 self-assigned this Jun 5, 2023
@dtrai2 dtrai2 marked this pull request as ready for review June 5, 2023 12:51
@dtrai2 dtrai2 marked this pull request as draft June 15, 2023 15:48
@ekneg54 ekneg54 added the refactoring only refactoring, no change in behavior label Oct 7, 2023
@ekneg54 ekneg54 marked this pull request as ready for review June 24, 2024 08:52
@ekneg54 ekneg54 merged commit 1fec611 into main Jun 24, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring only refactoring, no change in behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants