Skip to content
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

[Docs] add note and links for coercing primitive values #408

Merged
merged 1 commit into from
Apr 18, 2021

Conversation

DesignByOnyx
Copy link
Contributor

This should hopefully lay to rest #91

@codecov
Copy link

codecov bot commented Apr 9, 2021

Codecov Report

Merging #408 (feb3cb5) into master (dd0f954) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #408   +/-   ##
=======================================
  Coverage   99.78%   99.78%           
=======================================
  Files           8        8           
  Lines        1385     1385           
  Branches      167      167           
=======================================
  Hits         1382     1382           
  Misses          3        3           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd0f954...feb3cb5. Read the comment docs.

README.md Outdated
If you wish to auto-convert values which look like numbers, booleans, and other values into their primitive values, you can do one of the following:

- [Install Express JS middleware to auto-convert request query parameters](https://github.com/xpepermint/query-types)
- [Copy the above MIT licensed coercion code](https://github.com/xpepermint/query-types/blob/master/index.js) directly into your project and modify it as you see fit. This code has been used on some very large projects and gives you the greatest amount of control over your data.
Copy link
Owner

Choose a reason for hiding this comment

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

let's not advise people to copypasta code; if they want to use that code, they should depend on the package.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done - thanks for the feedback.

@ljharb ljharb force-pushed the auto-coercion-docs branch from feb3cb5 to c2ae487 Compare April 18, 2021 21:33
@ljharb ljharb changed the title docs: add note and links for coercing primitive values [Docs] add note and links for coercing primitive values Apr 18, 2021
@ljharb ljharb merged commit c2ae487 into ljharb:master Apr 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants