You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Throughout k6 we use camelCase for JSON options and CAPS_SNAKE_CASE for env variables. The few places where there are argline options such as k6 run -o csv=file_name=something.csv we sometimes use snake_case as in the CSV output and one option in the influxdb one, but sometimes we use camelCase as in the loki logs and the rest of the influxdb options :(.
Given that there really isn't a good case for one of the options to be camelCase and the other snake_case we should make them the same. For backwards compatible reasons we should still parse the snake_case variant and warn the user through a log message, at least for ... some versions.
So I propose that @codebien fixes the influxdb one in #2110 and there to be a separate PR for the CSV output.
The text was updated successfully, but these errors were encountered:
Go for it @Sampriti-Mitra ! Just FYI if you want to do it as part of hacktoberfest, you might need to wait another 3 days or so, although maybe it's fine, I haven't really read the fine print 🤷♂️
Throughout k6 we use camelCase for JSON options and CAPS_SNAKE_CASE for env variables. The few places where there are argline options such as
k6 run -o csv=file_name=something.csv
we sometimes use snake_case as in the CSV output and one option in the influxdb one, but sometimes we use camelCase as in the loki logs and the rest of the influxdb options :(.Given that there really isn't a good case for one of the options to be camelCase and the other snake_case we should make them the same. For backwards compatible reasons we should still parse the snake_case variant and warn the user through a log message, at least for ... some versions.
So I propose that @codebien fixes the influxdb one in #2110 and there to be a separate PR for the CSV output.
The text was updated successfully, but these errors were encountered: