Revert "feat(unstable): enable importsNotUsedAsValues by default (#7413)"

[nayeemrmn]

This reverts commit fbb18d4.

Ref #7413.

As discussed, the --no-check incompatiblity we were seeing with import { MyType } from "./file.ts"; was a symptom of swc-project/swc#1065 which is now fixed. The rule is no longer was never really needed.

[bartlomieju]

LGTM

[lucacasonato]

LGTM - before merging I want to confirm that this case gets stripped correctly now:

import { MyType } from "./foo.ts";
export type { MyType };

[nayeemrmn]

before merging I want to confirm that this case gets stripped correctly now:

import { MyType } from "./foo.ts";
export type { MyType };

Actually, just after opening this I realised the re-export thing was still broken and fixed it in swc: swc-project/swc#1126. Reiterating that I don't think these temporary swc bugs justify imposing rules for type checking so it breaks there, too. That's for fundamental incompatibilities, like how isolatedModules is needed for an efficient --no-check.

[nayeemrmn]

Also I think this change and that swc fix will both release at our next version.

[lucacasonato]

Yeah lets merge this then. As it is fixed upstream we will get the fix in Deno before 1.4.5 is released.