Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

watchify and browserify version upgraded #17

Closed
wants to merge 1 commit into from
Closed

watchify and browserify version upgraded #17

wants to merge 1 commit into from

Conversation

ayusharma
Copy link

ref #15

Configuration:

NPM : 3.6.0
Node JS : v5.7.0
MacOSx: OS X El Capitan v10.11.3

Error was due to old version of watchify. @4ndr01d3 debugging helped to solve this issue. :)

@daviddao daviddao changed the title watcify and browserify version upgraded watchify and browserify version upgraded Apr 4, 2016
@daviddao
Copy link
Member

daviddao commented Apr 4, 2016

Thanks! I think we should think about removing fixed versions and always take the latest dependencies to avoid this issue in the future.

@ayusharma
Copy link
Author

@daviddao Could we use "*" to download latest dependencies like :

"devDependencies": {
    "bootstrap": "*",
    "browserify": "*",
    "mkdirp": "*",
    "prunner": "*",
    "sniper": "*",
    "watchify": "*"
  },

@wilzbach
Copy link
Member

wilzbach commented Apr 4, 2016

That's a pretty bad idea considering the latest left-pad problem..

@ayusharma
Copy link
Author

@greenify I just read about left-pad problem. Surely, My approach is a hack to problem. How could be ensure that slush-biojs always install latest packages ? what could be the alternatives to remove fixed versions ?

I went through most of the slush repos and all have included the fixed version for dependencies.

@ayusharma ayusharma closed this Jul 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants