fixme icon indicating copy to clipboard operation
fixme copied to clipboard

Use camelCase instead of snake_case for options

Open kud opened this issue 6 years ago • 1 comments

Hello,

I know it's a tricky issue but I think the options should be in camelCase (https://eslint.org/docs/rules/camelcase)

In JavaScript:

  • lowerCamelCase for variables, properties, etc.
  • UpperCamelCase for constructors.
  • CAPITAL_CASE_WITH_UNDERSCORES for constants, which seem to be rarely used in JS anyway.
const fixme = require("fixme")

fixme({
  path: "./src/",
  ignored_directories: [],
  file_patterns: ["**/*"],
  file_encoding: "utf8",
  line_length_limit: 3000,
})

could be

const fixme = require("fixme")

fixme({
  path: "./src/",
  ignoredDirectories: [],
  filePatterns: ["**/*"],
  fileEncoding: "utf8",
  lineLengthLimit: 3000,
})

kud avatar Oct 15 '19 10:10 kud

I think this will just cause confusion. Also, programs rarely use camelCase for command line args.

jack-davidson avatar Jun 16 '21 13:06 jack-davidson