Skip to content

Comments

Configured Object.freeze and memoize to be excluded from side-effects rule#217

Merged
jpolavar merged 1 commit intomainfrom
configure-no-side-effects-object-freeze
Mar 27, 2025
Merged

Configured Object.freeze and memoize to be excluded from side-effects rule#217
jpolavar merged 1 commit intomainfrom
configure-no-side-effects-object-freeze

Conversation

@jpolavar
Copy link
Contributor

Closes #206

@jpolavar jpolavar added the PATCH label Mar 24, 2025
@jpolavar jpolavar requested a review from carlansley March 24, 2025 22:16
@jpolavar jpolavar self-assigned this Mar 24, 2025
@github-actions
Copy link

Beta Published - Install Command: npm install @checkdigit/eslint-config@11.2.2-PR.217-5b56

@github-actions
Copy link

✅ PR review status - All reviews completed and approved!

@jpolavar jpolavar merged commit 6ee1e46 into main Mar 27, 2025
8 checks passed
@jpolavar jpolavar deleted the configure-no-side-effects-object-freeze branch March 27, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Object.freeze(), assuming the argument is side-effect free, should be excluded from @checkdigit/no-side-effects rule

2 participants