Skip to content

Correct JSDoc for stateless escapeAll and quoteAll#2387

Merged
ericcornelissen merged 3 commits intomainfrom
fix-js-docs
Mar 4, 2026
Merged

Correct JSDoc for stateless escapeAll and quoteAll#2387
ericcornelissen merged 3 commits intomainfrom
fix-js-docs

Conversation

@ericcornelissen
Copy link
Copy Markdown
Owner

@ericcornelissen ericcornelissen commented Mar 4, 2026

Summary

Correct the description in the JSDoc for escapeAll and quoteAll in the stateless module. It incorrectly stated that non-array inputs are converted to one-element arrays (which was the behavior in v1 of this library). It now states that these are rejected explicitly, which has been added to the stateful variants' documentation too for clarity.

This problem was found by Codex.

Correct the description in the JSDoc for `escapeAll` and `quoteAll` in
the stateless module. It incorrectly stated that non-array inputs are
converted to one-element arrays (which was the behavior in v1 of this
library). It now states that these are rejected explicitly, which has
been added to the stateful variants' documentation too for clarity.
@ericcornelissen ericcornelissen added the documentation Improvements or additions to documentation label Mar 4, 2026
@deepsource-io

This comment was marked as off-topic.

@ericcornelissen

This comment was marked as resolved.

@coderabbitai

This comment was marked as spam.

@coderabbitai

This comment was marked as spam.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4eaa4dee-9ae7-4111-a60c-51e6e39d0af5

📥 Commits

Reviewing files that changed from the base of the PR and between 0a9a180 and 2771a0c.

📒 Files selected for processing (2)
  • src/index.js
  • src/stateless.js

Comment thread src/stateless.js
@ericcornelissen ericcornelissen merged commit 15f42c1 into main Mar 4, 2026
42 checks passed
@ericcornelissen ericcornelissen deleted the fix-js-docs branch March 4, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant