🐛 rome format ignores command line options

[MichaReiser]

Environment information

Version: `main`

What happened?

Run rome with quote-style="single"

cargo run --bin rome --release format ~/Desktop/jquery.min.js --quote-style=single

Rome formats the string literals in jquery.min.js with double quotes.

Expected result

The command line arguments take precedence over the configuration and Rome formats the string literals in jquery.min.js with single quotes

Code of Conduct

• I agree to follow Rome's Code of Conduct

[MichaReiser]

The problem seems to be in the CLI/Workspace as @leops pointed out in #3167 (comment)

I verified that quickly by adding a dbg!(&options) to format_node in rome_js_formatter and can confirm that the options use QuoteStyle::Double instead of single

[jfeltesse-mdsol]

I might be missing something but here it stills insists on using double quotes

rome -v
Rome CLI v0.9.1-next

cat rome.json
{
  "formatter": {
    "indentStyle": "space",
    "indentSize": 2,
    "lineWidth": 120
  },
  "linter": {
    "rules": {
      "ts": {
        "recommended": true
      }
    }
  }
}

rome format --quote-style single

# diffs like
# 0  | - import React from 'react';
#  0 | + import React from "react";

note: I'm trying to format .tsx files.

[hustcer]

I have met the same issue: rome v0.9.1 format with single quote does not work, maybe we should reopen this

[ematipico]

I was able to replicate the issue. I am going to re-open the issue and fix it