microsoft / microsoft/TypeScript

Incorrect type declarations for a constant inside a namespace merged with an enum

Ouverte
#61,944 1 commentaire 2 réactions 1 personne assignée Voir sur GitHub

Personne n'a encore pris cette issue.

Bug Domain: Declaration Emit Help Wanted
Langage dominant
Go
Étoiles
111k
Forks
14.3k
Merge moyen
2 j 4 h
PR mergées (30 j)
132

Description

### 🔎 Search Terms

"enum", "namespace", "declaration", "d.ts", "const"

### 🕗 Version & Regression Information

- This changed between versions 3.1.0-dev.20180906 and 3.1.0-dev.20180907

### ⏯ Playground Link

https://www.typescriptlang.org/play/?#code/KYOwrgtgBAYg9nKBvAUFdUBGBDATigXxRGwmAGcAHbAY2FgWTQ2AA9K5cAXKGuEcjxwAvKAF4GcAHQ5cAbkIogA

### 💻 Code

```ts
enum Foo {
bar
}
namespace Foo {
export const baz = Foo.bar;
}
```

### 🙁 Actual behavior

Typescript generates invalid d.ts for the enum value assigned to the constant (in all versions starting 3.1.0-dev.20180907)
```ts
declare enum Foo {
bar = 0
}
declare namespace Foo {
const baz = bar; // <-- unknown `bar` identifier, `baz` type is `any`
}
```
In versions before 3.1.0-dev.20180906 the constant is declared as `const baz: Foo;`

### 🙂 Expected behavior

Typescript generates a valid declaration of the constant
```ts
declare enum Foo {
bar = 0
}
declare namespace Foo {
const baz = Foo.bar;
}
```
or
```ts
declare enum Foo {
bar = 0
}
declare namespace Foo {
const baz: Foo;
}
```

### Additional information about the issue

_No response_

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.