bengreenier / bengreenier/overlayed
refactor object spreading logic
- Dominant language
- TypeScript
- Stars
- 28
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
per [this review comment](https://github.com/bengreenier/overlayed/pull/32#discussion_r222395313) there is a better way to do object spreading that is being used to extend an existing object with additional properties.
For instance:
```
const existing = { a: 'ayy' }
const modified = {...existing, b: 'bee' }
```
as opposed to
```
const existing = { a: 'ayy' }
const modified = {...existing, ...{ b: 'bee' } }
```
this should be done across the codebase at once, for consistency.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reviewing the linked pull-request comment and searching the codebase for object spreads that wrap additional properties in a second object. Replace each matching pattern with the direct property form, then verify that all such occurrences are consistent and the existing checks still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100