Skip to content

Conversation

@c-bryant
Copy link

In addition, updated the Config example section of the README to include an example of the watch property and it's intended use for clarity.

c-bryant added 2 commits July 30, 2019 00:17
…ence. In addition, updated the Config example section of the README to include an example of the watch property and it's intended use for clarity.
"dest": "bundled.scss"
"entry": "./src/main.scss",
"dest": "./bundled.scss",
"watch": "./src/*.scss"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added watch here to help clarify it's intended use as a PATH to watch, not a watch flag.

| --includePaths | includePaths | array | Include paths for resolving imports | | |
| --project, -p | project | string | Project location, where `node_modules` are located. | | |
| --ignoredImports | ignoredImports | array | Ignore resolving import content by matching a regular expression. | | |
| --watch, -w | watch | boolean | Watch files for changes. | | |
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed this duplicate & incorrect --watch argument reference that appears to have snuck in a while back. Pretty sure this wasn't intended to ever make it into the readme.

@c-bryant c-bryant marked this pull request as ready for review July 30, 2019 04:32
@c-bryant c-bryant changed the title Update README to remove incorrect --watch argument reference Update Readme: Remove incorrect '--watch' reference Jul 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants