-
Notifications
You must be signed in to change notification settings - Fork 227
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
Document the csv parser's asObjects
option
#1850
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On this page https://grafana.com/docs/k6/latest/javascript-api/k6-experimental/ the csv
link redirects to the wrong page. Can you fix it, please? 🙏
docs/sources/k6/next/javascript-api/k6-experimental/csv/Options.md
Outdated
Show resolved
Hide resolved
Good catch @codebien I've fixed the link 🙇🏻 |
@oleiade Can you fix the broken links on the previous versions as well or at least on the latest online, please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
docs/sources/k6/next/javascript-api/k6-experimental/csv/parse.md
Outdated
Show resolved
Hide resolved
docs/sources/k6/next/javascript-api/k6-experimental/csv/Parser.md
Outdated
Show resolved
Hide resolved
docs/sources/k6/next/javascript-api/k6-experimental/csv/Options.md
Outdated
Show resolved
Hide resolved
Co-authored-by: İnanç Gümüş <[email protected]>
Fixed the broken links down to version 0.54 where the csv module was introduced. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left just one tiny suggestion, LGTM! 🙇
docs/sources/k6/next/javascript-api/k6-experimental/csv/Options.md
Outdated
Show resolved
Hide resolved
@@ -4,7 +4,7 @@ title: javascript-api/k6-experimental | |||
|
|||
| Modules | Description | | |||
| ------------------------------------------------------------------------------------------------ | -------------------------------------------------------------------------------------------------------------------------- | | |||
| [csv](https://grafana.com/docs/k6/<K6_VERSION>/javascript-api/k6-experimental/fs) | Provides support for efficient and convinient of parsing CSV files. | | |||
| [csv](https://grafana.com/docs/k6/<K6_VERSION>/javascript-api/k6-experimental/csv) | Provides support for efficient and convenient parsing of CSV files. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this. 🙇
…s.md Co-authored-by: Heitor Tashiro Sergent <[email protected]>
What?
Documents the newly introduced
asObjects
option of thek6/experimental/csv
module.Checklist
npm start
command locally and verified that the changes look good.docs/sources/k6/next
folder of the documentation.Related PR(s)/Issue(s)
grafana/k6#4295