fix: reject unsupported URL schemes#604
Conversation
|
✅ DCO Check Passed Thanks @PragnyaKhandelwal, all your commits are properly signed off. 🎉 |
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🔴 Require two reviewer for test updatesWaiting for
This rule is failing.When test data is updated, we require two reviewers
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
|
Signed-off-by: Pragnya <prag1704@gmail.com>
Signed-off-by: Pragnya <prag1704@gmail.com>
d78b7be to
ee3fef7
Compare
|
Hi team! Just a gentle nudge on this PR as the new week starts. It addresses the URL scheme crash in #3396 and includes regression tests. I noticed the GitHub Actions are currently awaiting maintainer approval to run—would appreciate it if someone could approve the workflows so the CI can verify the changes! Thanks! |
|
Thanks for the approval, @sasi22498! |
Summary
Linked issue
Fixes docling-project/docling#3396
Testing
pytest test/test_utils.py