You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The source file jspm-cli/lib/config.js appears to relate to two separate concerns. It deals with setting values into the global config file ~/.jspm/config and well as working with a project's config.js file. Unless these two concerns are intertwined, it might be better to break the file into two. This would help new developers to the project to better understand what each file is doing.
The text was updated successfully, but these errors were encountered:
Definitely agreed that this file needs to be better structured (perhaps into lib/project.js for the package.json and lib/config.js for the global config and config helpers). The code also needs to be cleaned up as well.
The source file
jspm-cli/lib/config.js
appears to relate to two separate concerns. It deals with setting values into the global config file~/.jspm/config
and well as working with a project'sconfig.js
file. Unless these two concerns are intertwined, it might be better to break the file into two. This would help new developers to the project to better understand what each file is doing.The text was updated successfully, but these errors were encountered: