SAFE-Stack / SAFE-template

dotnet CLI template for SAFE project
MIT License
280 stars 87 forks source link

Add exclude files to settings.json #617

Open isaacabraham opened 2 weeks ago

isaacabraham commented 2 weeks ago

We should consider excluding folders from VS Code. This can be included in the settings.json file:

    "files.exclude": {
        ".vs": true,
        ".npmrc": true,
        ".paket": true,
        "**/bin": true,
        "**/obj": true,
        "**/output": true,
        "deploy": true,
        "node_modules": true,
        "paket-files": true,
        ".fantomasignore": true,
        "src/Client/postcss.config.js": true,
        "src/Client/vite.config.mts": true,
        "src/Client/tailwind.config.js": true,
        "src/Client/index.css": true,
        "src/Client/index.html": true,
    }

This is just an idea - the list above might be too much though. Certainly the first half of the list would be good to add - the second half might be more controversial ;-)

Before:

image

After:

image

Larocceau commented 2 weeks ago

I agree on excluding the first part of the list, second part not so much. I feel like those files tend to be a source of issues that can be fairly hard to pinpoint because they are in config files. For people not as familiar with the template hiding part of the config can make troubleshooting more challenging that necessary.

jwthomson commented 1 week ago

@isaacabraham what are your thoughts on Maxime's preferred solution to this problem - nesting config under a single file using this extension: https://github.com/antfu/vscode-file-nesting-config