search

standard / standard-engine

:fire_engine: The guts of `standard` modularized for reuse
MIT License
145 stars 39 forks source link

fix(cli): warning message mention homepage #268

Closed theoludwig closed 3 years ago

theoludwig commented 3 years ago

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update [x] Bug fix [ ] New feature [ ] Other, please explain:

What changes did you make? (Give an overview)

standard-engine is being used not only for standard itself but also for other engines like ts-standard, happiness, etc.

That being said, it makes sense to not include specific things related to standard only. This PR remove the mention of standardjs.com in a warning message.

voxpelli commented 3 years ago

Maybe replace it with the content of opts.homepage if there is one set? Else leave it out like you did now?

theoludwig commented 3 years ago

Maybe replace it with the content of opts.homepage if there is one set? Else leave it out like you did now?

Good idea! :smile: Done, it's updated. @voxpelli