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

Dependency management #16

Closed
sgdavis1 opened this issue May 31, 2018 · 4 comments
Closed

Dependency management #16

sgdavis1 opened this issue May 31, 2018 · 4 comments
Assignees

Comments

@sgdavis1
Copy link
Member

We currently have several outdated packages that we depend on, and should consider upgrading. These (sqlite3, fs-extra, gulp-sass) are major version updates. We will need to ensure that when these are upgraded, none of the code relying on these breaks due to any possible API changes.

Please upgrade the package.json file to allow these major version updates and then perform the upgrade and fully test it. Double check the documentation on those modules to see if we have to make any changes in our code to keep up with their best practices.

@sgdavis1
Copy link
Member Author

Also while doing this, let's re-write our package.json to drop any patch level specified for better compatibility with what we intend.

Ex]
Change: "sequelize": "^4.37.7" to "sequelize": "^4.37"

@osandvold302
Copy link
Contributor

The latest version of Gulp, 4.0.0, removes the list parameter syntax to define tasks and task dependencies. They have provided a boilerplate to model your project's gulpfile on -- https://github.com/jh3y/gulp-boilerplate. They also introduce series() and parallel() to clearly define dependencies and order of operations

@osandvold302
Copy link
Contributor

With the depreciation of gulp-util, packages have been migrating to requiring Gulp 4.0.0. Gulp-sass v4.0.0 has dropped support see here. Gulp-livereload has yet to drop dependency on util, Issue here. Gulp-nodemon has also updated dependencies (just yesterday!) to Gulp 4.0.0 and dropped support for gulp-utils see here.

@osandvold302
Copy link
Contributor

Complete with dependencies branch merged into devel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants