Last 12 weeks · 0 commits
4 of 6 standards met
Description This PR addresses the TODO comment in the codebase by replacing the use of with a simpler loop in the function. Changes Replaced with a loop for better readability Removed the eslint-disable comment for Maintained the same functionality Why The TODO comment indicated that should be removed. This refactor makes the code: Easier to understand More performant (no intermediate array creation) Consistent with modern JavaScript patterns Testing The existing test suite should cover this functionality. Please run to verify.
Repository: sindresorhus/query-string. Description: Parse and stringify URL query strings Stars: 6906, Forks: 453. Primary language: JavaScript. Languages: JavaScript (94.7%), TypeScript (5.3%). License: MIT. Topics: npm-package, parse, query-string, stringify, url, urlsearchparams. Latest release: v9.3.1 (5mo ago). Open PRs: 1, open issues: 1. Last activity: 5mo ago. Community health: 85%. Top contributors: sindresorhus, SamVerschueren, scottenock, Richienb, dubzzz, viczhuravlev, cheqianxiao, bug-brain, rapzo, kevva and others.
JavaScript
In version 7.1.3 I can import stringify now in version ^8 a I can't, and when I try to refactor the code to import queryString and use queryString.stringify my tests with jest broken like that: `