Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(README): clarify path option description (options.config.path) #371

Merged

Conversation

y-gagar1n
Copy link
Contributor

The description for path option in README.md is a bit misleading. It says PostCSS Config Path while in fact it sets only the directory where postcss.config.js will be looked for.

It leads to a confusion, because users may think that they can set arbitrary name for their config file.

Type


  • Docs

SemVer


  • Bug (:label: Patch)

Issues


Checklist


  • Lint and unit tests pass with my changes
  • I have added tests that prove my fix is effective/works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes are merged and published in downstream modules

@coveralls
Copy link

coveralls commented Jul 14, 2018

Coverage Status

Coverage remained the same at 80.233% when pulling 5e3aa5d on y-gagar1n:docs-config-path-directory into 4955792 on postcss:master.

@michael-ciniawsky michael-ciniawsky changed the title Clarify 'path' option description in docs docs(README): clarify 'path' option description (options.config.path) Jul 14, 2018
README.md Outdated
|[`context`](#context)|`{Object}`|`undefined`|PostCSS Config Context|

#### `Path`

You can manually specify the path to search for your config (`postcss.config.js`) with the `config.path` option. This is needed if you store your config in a separate e.g `./config || ./.config` folder.

> ⚠️ Otherwise it is **unnecessary** to set this option and is **not** recommended
> ⚠️ Otherwise it is **unnecessary** to set this option and is **not** recommended. You **can't** set other name for **postcss.config.js**, this option only allows you to specify the folder where it will be looked for.
Copy link
Member

Choose a reason for hiding this comment

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

⚠️ Note that you can't use other filename other then the supported config formats (e.g .postcssrc.js, postcss.config.js), this option only allows you to manually specify the directory where config lookup should start from

webpack.config.js

{
   config: {
      path: 'path/to/.config/'  
      path: 'path/to/.config/css.config.js'  
   }
}

@y-gagar1n
Copy link
Contributor Author

Fixed

README.md Outdated
@@ -144,20 +144,23 @@ If you use JS styles without the [`postcss-js`][postcss-js] parser, add the `exe

You can manually specify the path to search for your config (`postcss.config.js`) with the `config.path` option. This is needed if you store your config in a separate e.g `./config || ./.config` folder.

> ⚠️ Otherwise it is **unnecessary** to set this option and is **not** recommended. You **can't** set other name for **postcss.config.js**, this option only allows you to specify the folder where it will be looked for.
> ⚠️ Note that you **can't** use a **filename** other than the [supported config formats] (e.g `.postcssrc.js`, `postcss.config.js`), this option only allows you to manually specify the **directory** where config lookup should **start** from
Copy link
Member

Choose a reason for hiding this comment

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

Please still leave the previous warning

⚠️ Otherwise it is unnecessary to set this option and is not recommended.

@y-gagar1n
Copy link
Contributor Author

Fixed

Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

@michael-ciniawsky michael-ciniawsky changed the title docs(README): clarify 'path' option description (options.config.path) docs(README): clarify path option description (options.config.path) Jul 16, 2018
@michael-ciniawsky michael-ciniawsky merged commit 5c1c108 into webpack-contrib:master Jul 16, 2018
@y-gagar1n
Copy link
Contributor Author

Turns out those ✅ and ❌ symbols are shown as squares on Chrome in Debian.

@michael-ciniawsky
Copy link
Member

Can you post a screenshot and does this also happen with other symbols (e.g the various ⚠️) ? Chrome only or in other browsers aswell ?

@y-gagar1n
Copy link
Contributor Author

Hmm, must've been some specific version of Chrome or something. Seen that on my work machine today. Now I check on my personal machine (also Chrome and Debian) and everything is OK. Will discover it more tomorrow.

@y-gagar1n
Copy link
Contributor Author

It's caused by the lack of some system fonts on my Debian. I've made a little research and found out that there are more widely supported symbols for 'cross' and 'check' marks. The downside is that they are not so fancy looking.

Here they are with their supported fonts:

Used now:

http://www.fileformat.info/info/unicode/char/2705/fontsupport.htm
http://www.fileformat.info/info/unicode/char/274c/fontsupport.htm

Recommended:

https://www.fileformat.info/info/unicode/char/2714/fontsupport.htm
https://www.fileformat.info/info/unicode/char/2716/fontsupport.htm

@y-gagar1n
Copy link
Contributor Author

And here is the screenshot you asked for. Same behavior in Chrome and Opera.

Firefox works fine.

2018-07-17 11-32-59

@michael-ciniawsky michael-ciniawsky removed this from Docs in Dashboard Aug 4, 2018
@michael-ciniawsky michael-ciniawsky removed this from the 3.0.1 milestone Aug 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants