-
Notifications
You must be signed in to change notification settings - Fork 761
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider replacing qs
with URLSearchParams
#3585
Labels
Comments
Started the effort in #3633 |
char0n
added a commit
that referenced
this issue
Aug 11, 2024
char0n
added a commit
that referenced
this issue
Aug 11, 2024
@wojtekmaj addressed in #3634. I would appreciated your review. |
char0n
added
version: 3.x
type: enhancement
dependencies
Pull requests that update a dependency file
labels
Aug 11, 2024
char0n
added a commit
that referenced
this issue
Aug 12, 2024
char0n
added a commit
that referenced
this issue
Aug 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Is your feature request related to a problem?
qs
is pulling in 14 other dependencies with it: https://npmgraph.js.org/[email protected], making swagger-js installing, evaluating and running slower than it could. On top of that, since qs is being maintained by you-know-who, changes are that even more needless dependencies will pop up in the dependency tree.Describe the solution you'd like
Consider replacing
qs
withURLSearchParams
.Describe alternatives you've considered
Custom function like:
if nested value support would be needed.
The text was updated successfully, but these errors were encountered: