GeezerVison

Hi Abubaker Bashir (@AbubakerB), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, MSBOT;

For this to work properly, the visibility needs to be part of the construct signature rather than being a declaration-based check that happens during signature resolution. Otherwise it's going to be too difficult to enforce assignability constraints based on constructor visibility.

A relevant testcase to add would be:

class Foo {
  private constructor() { }
}
class Bar {
  protected constructor() { }
}
let x: new() => any;
x = Foo; // Should be an error
x = Bar; // Should be an error

Ryan Cavanaugh (@RyanCavanaugh) SignaturesRelatedTo now checks visibility of constructor.

Should i close this off? or is this in the pipeline?

The declaration emitter will need to be amended - add // @declaration: true to the top of every affected test file.

The following test cases would be useful too:

class Base {
    protected constructor() {
    }
}

class Derived extends Base {
    private constructor() {
        super();
    }
}

class Base {
    protected constructor() {
    }
}

class Derived extends Base {
    protected constructor() {
        super();
    }
}

var baseCtor = Base;
baseCtor = Derived;

Abubaker Bashir (@AbubakerB) sorry this fell off of our radar, we've been hard at work on TypeScript 1.6.

Anders Hejlsberg (@ahejlsberg) Mohamed Hegazy (@mhegazy) Vladimir Matveev (@vladima) any thoughts?

Daniel Rosenwasser (@DanielRosenwasser) I see, that's fine :)

1. I've added // @declaration: true to the three test files.
2. Added test cases under classConstructorAccessibility2.ts. Although, I have one question... Can a X ctor signature be assigned to a non-X ctor signature? (where X is public/protected/private).
3. Renamed the diagnostic message for error code 2656 to be more clear.

Abubaker Bashir (@AbubakerB) can you get this up-to-date and resubmit in a new PR? Very sorry we left this hanging for so long.