typescript-go

typescript-go copied to clipboard

runner: Strada emits positions of some baselines as `--`; Corsa doesn't

1

defaultPropsEmptyCurlyBecomesAnyForJs, discriminantNarrowingCouldBeCircular

sandersn

node builder: Strada's `(Missing)` is baselined as the empty string in Corsa

1

defaultKeywordWithoutExport2

sandersn

runner: Strada baselines rarely put `./` on front of filename, but Corsa always removes it

1

decoratorMetadataTypeOnlyExport

sandersn

module resolution: missing a decl on imported type alias

2

declarationEmitBundleWithAmbientReferences

sandersn

program: `<reference no-default-lib="true"/>` is ignored

1

declarationEmitBundlePreservesHasNoDefaultLibDirective

sandersn

In #241, we had quite a bit of discussion about whether certain fields belong to `System` vs `Host` vs `Program` layering, with all reviewers agreeing that the current setup "feels...

iisaduan

Other differences probably exist too. See tests/cases/conformance/jsx/jsxInvalidEsprimaTestSuite.tsx tests/baselines/reference/constructorWithIncompleteTypeAnnotation.js tests/baselines/reference/contextuallyTypedJsxAttribute.js I observed an attribute `as` parse as a type assertion, for example: _submodules/TypeScript/tests/baselines/reference/contextuallyTypedJsxAttribute.js

sandersn

Not sure if it's not parsed or just not walked. See tests/cases/conformance/expression/superCalls/errorSuperCalls.ts

sandersn

See _submodules/TypeScript/tests/cases/conformance/classes/members/privateNames/privateNameHashCharName.ts

sandersn

I think the feature is still missing and ImportAttributes is just what the parser comes up with currently. See tests/cases/conformance/importAssertion/importAssertion1.ts

sandersn