Implement From<LocalName> on QName

[Tpt]

No description provided.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.15%. Comparing base (e00ae5c) to head (e4c5519).
⚠️ Report is 74 commits behind head on master.

Files with missing lines	Patch %	Lines
src/name.rs	0.00%	3 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1015      +/-   ##
==========================================
- Coverage   57.31%   55.15%   -2.16%     
==========================================
  Files          46       51       +5     
  Lines       18197    18794     +597     
==========================================
- Hits        10429    10366      -63     
- Misses       7768     8428     +660     

Flag	Coverage Δ
unittests	55.15% <0.00%> (-2.16%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

[dralley]

This looks fine. Did you want to implement this suggestion, or do you think it's fine without it?

It seems that it will be even safe to add impl PartialEq<Rhs = QName> for LocalName and impl PartialEq<Rhs = LocalName> for QName, so you may directly check equivalence of local names and qualified names.

[dralley]

I think you meant #1015?

[Tpt]

Thank you! Fixed.

[Tpt]

@dralley I would tend to think it's fine without it but glad to add it in an other MR if you want