Use `unknown` instead of `any` for `StringIndexed`
Open
Nobody has claimed this yet.
discussion
semver:major
TypeScript-specific
- Dominant language
- TypeScript
- Stars
- 2.9k
- Forks
- 445
- Avg merge
- 1d 3h
- Merged PRs (30d)
- 28
Description
As per title, I think it should be unknown to prevent accessing properties which may not exist.
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
Start by reading src/types/helpers.ts at line 5, the location linked in the issue, and inspect how StringIndexed is used. Update the type as requested and confirm the existing TypeScript checks still pass without allowing unsupported property access.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- developer-experience
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100