Skip to content

Add no-error-capture-stack-trace rule#2676

Merged
sindresorhus merged 15 commits into
sindresorhus:mainfrom
fisker:no-error-capture-stack-trace
Jun 13, 2025
Merged

Add no-error-capture-stack-trace rule#2676
sindresorhus merged 15 commits into
sindresorhus:mainfrom
fisker:no-error-capture-stack-trace

Conversation

@fisker

@fisker fisker commented Jun 12, 2025

Copy link
Copy Markdown
Collaborator

Fixes #2627

@fisker fisker marked this pull request as ready for review June 12, 2025 15:53
@fisker fisker marked this pull request as draft June 12, 2025 15:54
@fisker fisker marked this pull request as ready for review June 12, 2025 15:56
@sindresorhus

Copy link
Copy Markdown
Owner

Should it be no-useless-error-capture-stack-trace? Since it doesn't prevent all uses of Error.captureStackTrace, just ones inside error subclass.

@sindresorhus sindresorhus merged commit fc900b6 into sindresorhus:main Jun 13, 2025
19 checks passed
@fisker fisker deleted the no-error-capture-stack-trace branch June 13, 2025 12:22
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.

Rule proposal: no-error-capture-stack-trace

2 participants