Skip to content

Excluded Object.freeze() from no-side-effects#215

Closed
jpolavar wants to merge 7 commits intomainfrom
no-side-effects-object-freeze
Closed

Excluded Object.freeze() from no-side-effects#215
jpolavar wants to merge 7 commits intomainfrom
no-side-effects-object-freeze

Conversation

@jpolavar
Copy link
Contributor

Closes #206

@jpolavar jpolavar added the PATCH label Mar 12, 2025
@jpolavar jpolavar self-assigned this Mar 12, 2025
@github-actions
Copy link

Beta Published - Install Command: npm install @checkdigit/eslint-config@11.2.2-PR.215-7c39

@jpolavar jpolavar closed this Mar 24, 2025
@ramaghanta ramaghanta deleted the no-side-effects-object-freeze branch May 27, 2025 19:13
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

1 participant