refactor(io): deprecate types.d.ts

[iuioiua]

Reader, ReaderSync, Writer and WriterSync interfaces are now supported in Deploy.

FYI types.d.ts was added in #912, in May 2021.

[ayame113]

I thought this was part of the work being done on denoland/deno#9795, as #912 (comment) says. maybe we don't need to remove this? (sorry if i'm wrong)

[iuioiua]

Hm... Perhaps, the opposite should be done. Instead of removing types.d.ts, it should be extended, so all Deno.Reader/Deno.Writer instances are replaced with these generic types.

There may be reasons to either entirely deprecate this file or fully adapt it throughout the rest of the repo.

Thoughts?

[kt3k]

I'm in favor of the opposite of this PR. As @ayame113 points, we're deprecating and removing Deno.Reader etc in the future. These types in io/types.d.ts should be used everywhere in std instead of Deno.Reader etc.

[iuioiua]

Another benefit would be gaining web compatibility in some areas, as long as users implement their own Reader/Writer classes.

@bartlomieju, WDYT? If we all agree, I'd be happy to implement it.

[bartlomieju]

I'm in favor of the opposite of this PR. As @ayame113 points, we're deprecating and removing Deno.Reader etc in the future. These types in io/types.d.ts should be used everywhere in std instead of Deno.Reader etc.

I agree with that as well.

[iuioiua]

Closing in favour of #3030.