microsoft / microsoft/TypeScript
Bloomberg TS 5.6 Beta Feedback
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 111k
- Forks
- 14.4k
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 117
Description
Acknowledgement
- I acknowledge that issues using this template may be closed without further explanation at the maintainer's discretion.
Comment
We are in the process of evaluating the impact of TS 5.6 beta on Bloomberg code. Below are preliminary findings.
Overall, the only new diagnostics produced by this release are the excellent improvements to nullish/truthy checks.
| Change | Impacts | Release notes | Packages impacted |
|---|---|---|---|
| New nullish/truthy diagnostics | Type checker | Disallowed Nullish and Truthy Checks | ~one dozen |
Stricter type-checking of partial objects
We observed new errors related to nullish/truthy checks, all of which look like correct diagnostics. A few simplified examples are below:
const err = "bar";
// Right operand of ?? is unreachable because the left operand is never nullish.(2869)
const errorMessage = "foo " + err ?? "Unknown error";
// Right operand of ?? is unreachable because the left operand is never nullish.(2869)
const isActive = !someCondition ?? true;
// This kind of expression is always truthy.(2872)
if (myCheck === "foo" || "bar") {
}
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The report contains TypeScript 5.6 beta feedback and simplified diagnostics, but names no file, test, or entry point to investigate. Confirm the reported diagnostics against the linked release notes and identify a reproducible repository case before defining any code change.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- compilers
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100