Remove Unnecessary lib.*.d.ts Entries

[SlurpTheo]

Why in lib.es6.d.ts do these exist:

interface String {
    /** Returns a string representation of a string. */
    toString(): string;
}

interface Symbol {
    /** Returns a string representation of an object. */
    toString(): string;
}

Where this does not:

interface Boolean {
    /** Returns a string representation of a boolean. */
    toString(): string;
}

Cleanup redundant (inherited from Object) entries like toString(): string on some interfaces.

RE: https://github.com/Microsoft/TypeScript/pull/23721#issuecomment-388128341

[mhegazy]

PRs welcomed.

[thejohnfreeman]

Should this be closed? The file now has no declarations of its own.

[SlurpTheo]

Should this be closed? The file now has no declarations of its own.

So, lib.es6.d.ts doesn't have them anymore, but it "ninjas" back to using lib.es5.d.ts which still contains:

interface String {
    /** Returns a string representation of a string. */
    toString(): string;
}

interface Symbol {
    /** Returns a string representation of an object. */
    toString(): string;
}

[SlurpTheo]

When you remove these "needless" entries, you don't get .toString() showing up in IntelliSense (like is the case today on a boolean in VSCode@1.39.0)