search

dojo / cli-build-webpack

🚀 **DEPRECATED** Dojo 2 - cli command for building applications
http://dojo.io
Other
4 stars 19 forks source link

Currently needs git as a pre-req due to reliance on non-published module #76

Closed rishson closed 7 years ago

rishson commented 7 years ago

Can we either publish "source-map-loader": "bryanforbes/source-map-loader#463701b", or remove the dep for something else please?

At the moment, you cannot use the cli to bootstrap a project unless you have git installed and relying on non-publically-versioned things is not great.

dylans commented 7 years ago

Is source-map-loader something specific for Dojo 2 or a fork of another project?

rishson commented 7 years ago

@bryanforbes ?

kitsonk commented 7 years ago

In fact the commit that it is depending upon has been blatted from the @bryanforbes fork, so you can no longer install this package. 😭

rishson commented 7 years ago

..which is a perfect example of why non-published deps being bad.

rishson commented 7 years ago

I'd like this done for beta 1 because relying on a non npm dep is more error prone that I would like. I don't care if the dep is published under Dojo or not, just that it is published (so can be resurrected if the repo goes away, or the commit, or..)