Skip to content

Feature/rfc compliance#49

Open
mmucklo wants to merge 1 commit intomasterfrom
feature/rfc-compliance
Open

Feature/rfc compliance#49
mmucklo wants to merge 1 commit intomasterfrom
feature/rfc-compliance

Conversation

@mmucklo
Copy link
Owner

@mmucklo mmucklo commented Feb 8, 2026

No description provided.

@codecov
Copy link

codecov bot commented Feb 8, 2026

Codecov Report

❌ Patch coverage is 77.08333% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.00%. Comparing base (4ea8f3f) to head (3e2d68f).

Files with missing lines Patch % Lines
src/Parse.php 74.52% 40 Missing ⚠️
src/ParseOptions.php 81.92% 15 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #49      +/-   ##
============================================
- Coverage     77.02%   77.00%   -0.03%     
- Complexity      219      287      +68     
============================================
  Files             3        3              
  Lines           505      674     +169     
============================================
+ Hits            389      519     +130     
- Misses          116      155      +39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Replace hardcoded validation with configurable boolean rule properties
on ParseOptions. Add static factory presets: rfc5321() (strict ASCII),
rfc6531() (internationalized UTF-8 with NFC normalization and IDN),
rfc5322() (standard with obsolete syntax), and rfc2822() (max compat).

Includes quoted-string content validation, C0/C1 control character
rejection, domain literal validation, octet-based length enforcement,
and comprehensive test coverage via testspec.yml (~200+ assertions).
@mmucklo mmucklo force-pushed the feature/rfc-compliance branch from f0fc4db to 3e2d68f Compare February 17, 2026 07:02
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.

1 participant