Skip to content

fix: removes polyfills intended to fix issues with older browsers.#1029

Merged
sebastianthulin merged 1 commit intomasterfrom
fix/remove-polyfills
Apr 11, 2025
Merged

fix: removes polyfills intended to fix issues with older browsers.#1029
sebastianthulin merged 1 commit intomasterfrom
fix/remove-polyfills

Conversation

@sebastianthulin
Copy link
Copy Markdown
Member

Polyfill Needed? Notes
NodeList.prototype.forEach Only for IE11 Remove unless you support IE11
Array.prototype.forEach ❌ Not needed Safe to remove
Object.prototype.forEach ❌ Bad practice Never do this, use Object.entries() etc.

@sonarqubecloud
Copy link
Copy Markdown

@sebastianthulin sebastianthulin merged commit c6614fd into master Apr 11, 2025
4 checks passed
@sebastianthulin sebastianthulin deleted the fix/remove-polyfills branch April 11, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants