Breaking: Consistency across all settings

Nothing here is set in stone. Tagged with help wanted so I can get users thoughts and input

Please keep comments related to this issue. Want something else adding/sorting? Please open a new issue

---

Bought to light in #22. The new forceBaseDirectory setting is inconsistent with other settings in the extension. Upon reviewing the setting I think it would be handy to force consistency across all settings.

Issue

• New setting causing confusion as it conflicts with other settings in the extension
  • forceBaseDirectory = "path/to/folder/"
  • Others = "/path/to/folder"
• Other setting could have better pattern management for:
  • Consistency
  • Help keep the programme running at it’s optimal

Resolution All setting that relate to paths will have the same pattern:

• A leading slash
• No ending slash

The includeFiles setting, which relates to files, will have the following restrictions:

• A leading slash
• Must end in .sass or .scss

---

Documentation changes are required

Error detection change

try{ if (!fs.statSync(baseDirectory).isDirectory()) { /*err not folder*/ } } catch { /* err doesn't exist*/ }

---

Edit: Updated to reflect changes across the settings as a whole (not just the new one in v4.4)

[szegheo]

Should I change all patterns so they must start with a slash but not end with one?

That would be the best (my vote).

[glenn2223]

Fixed in rc2