Skip to content
This repository has been archived by the owner on Sep 21, 2023. It is now read-only.

docs: update fmt section in config file manual #682

Closed
wants to merge 1 commit into from

Conversation

ElGranTorino
Copy link

I refactored the config file for compatibility with the new Deno version. The changes include replacing the "fmt" section with the following structure:

"fmt": {
    "options": {
      "lineWidth": 80,
      "indentWidth": 4,
      "semiColons": false,
      "singleQuote": true,
      "proseWrap": "preserve"
    },
    "files": {
      "exclude": ["src/"],
      "include": ["src/testdata/", "data/fixtures/**/*.ts"]
    }
},

instead of

"fmt": {
    "useTabs": true,
    "lineWidth": 80,
    "indentWidth": 4,
    "semiColons": false,
    "singleQuote": true,
    "proseWrap": "preserve",
    "include": ["src/"],
    "exclude": ["src/testdata/", "data/fixtures/**/*.ts"]
},

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Andrew seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@kt3k
Copy link
Member

kt3k commented Jul 3, 2023

I don't see the motivation of this change well. The existing example uses newer forms of configuration, which were introduced in 1.33 ( https://deno.com/blog/v1.33#flatter-denojson-configuration )

@kt3k kt3k closed this Sep 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants