rule suggestion: no-internal-module

[vvakame]

microsoft/TypeScript#2159
TypeScript 1.5.3 released. it has namespace keyword.
internal module is confusion name. I think use namespace instead of internal module.

if I create the pull request, will you accept it?

[ashwinr]

yes please. let's call the rule name no-module-keyword.

[vvakame]

@ashwinr yeah! I have one question. external module syntax are required in es6 module context.
like this.

declare module "foobar" {
    export default function (): string;
}

can I use no-module-keyword ?

[ashwinr]

But that's a module declaration and not a module definition. We should only be forbidding module definitions using the module keyword. Maybe the rule name should be no-module-definition

[adidahiya]

how about no-internal-module? I think "module definition" may be confusing / require lookup in our documentation

EDIT: just realized that the issue title had the same name suggestion. so looks like we arrived at the same thing independently :)

[ashwinr]

sgtm.