Revert "Reimplement idna on top of ICU4X (#923)"

[valenting]

This reverts commit 3d6dbbb.

See #937 for reasons behind this backout.

[codecov]

Codecov Report

Attention: Patch coverage is 93.16940% with 25 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@dcfbed3). Learn more about missing BASE report.

Files	Patch %	Lines
idna/src/uts46.rs	94.31%	17 Missing ⚠️
idna/src/punycode.rs	90.00%	3 Missing ⚠️
idna/tests/uts46.rs	86.36%	3 Missing ⚠️
idna/src/lib.rs	66.66%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #942   +/-   ##
=======================================
  Coverage        ?   81.76%           
=======================================
  Files           ?       21           
  Lines           ?     3549           
  Branches        ?        0           
=======================================
  Hits            ?     2902           
  Misses          ?      647           
  Partials        ?        0           

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

[djc]

Might be easier to only revert the url -> idna dependency to 0.5 for now?

[valenting]

Might be easier to only revert the url -> idna dependency to 0.5 for now?

rust-url/url/Cargo.toml

Line 29 in 3d6dbbb

idna = { version = "1.0.0", path = "../idna" }

idna = { version = "1.0.0", path = "../idna" }

That might be a better idea, but we'd have to stop referring dependencies by path. 🤔