bengreenier / bengreenier/overlayed

refactor object spreading logic

Open
#36 0 comments 0 reactions 0 assignees View on GitHub
enhancement tooling
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.