ts-types-esm: Model.createBindingContext returns Context|undefined
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 25/100
- Issue type
- Bug
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- typescript
- Domain
- tooling
Research direction
Start by locating the TypeScript declaration and implementation for Model.createBindingContext, then trace how it calls resolve and handles a missing parent context. Compare the declared return type with the null or undefined behavior described in the issue. Done means the intended behavior and nullability are made consistent, with relevant tests updated or added if the project has coverage for this API.
Written by the indexing model from the issue text.
Description
This is not so much a jsdoc error as a weird / stupid behavior (imo)
createBindingContext can apparently return in some case undefined (technically null based on the implementation) however when writing code this will never happen except when you do some weird thing.
It implies that resolve returns undefined (which btw is not in the signature) which would only happen if you have a relative binding without a parent context defined...
So ok it could happen but you'd be really looking for trouble writing such code :)
I wonder if it really makes sense to return undefined at all or if you should have an unresolveable context of some kind in there somewhere.
- Dominant language
- TypeScript
- Stars
- 213
- Forks
- 35
- Avg merge
- 37m
- Merged PRs (30d)
- 1
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.
More from UI5/typescript
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
UI5/typescript#625 · 4 comments ·
-
dts-generator
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
UI5/typescript#529 · 3 comments ·
-
Dependency Dashboard Open
Difficulty 5/5 Over a week Newbie friendliness 15/100
UI5/typescript#587 ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
UI5/typescript#532 ·
-
enhancement ts-interface-generator
Difficulty 5/5 Over a week Newbie friendliness 30/100
UI5/typescript#524 ·
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
area:tools bug good first issue help wanted priority:P2
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
TaewoooPark/Motifcode#14 ·
-
bug 🐞
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
[Bounty proposal] fix(web): memory insights count an evening memory on the next day ($25 proposed) Open
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
BasedHardware/omi#15320 ·
-
Difficulty 2/5 Half a day Newbie friendliness 78/100
vercel/vercel-plugin#199 ·