Skip to content
This repository has been archived by the owner on Nov 4, 2023. It is now read-only.

Changing dependency for left-pad #2

Closed
wants to merge 3 commits into from
Closed

Changing dependency for left-pad #2

wants to merge 3 commits into from

Conversation

ccutch
Copy link

@ccutch ccutch commented Mar 22, 2016

Left pad deprecated version 0.0.3 and now is using version 0.0.9 . this is causing a dependency fault to babel core, changed version and all tests passed.

@ccutch
Copy link
Author

ccutch commented Mar 22, 2016

changed dependency to now point at lodash instead of left-pad. This is a more trusted library

@@ -1,7 +1,7 @@
// Copyright 2014, 2015 Simon Lydell
// X11 (“MIT”) Licensed. (See LICENSE.)

var leftPad = require("left-pad")
var padStart = require("lodash").padStart
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe instead of the whole library, just this -> https://www.npmjs.com/package/lodash.padstart

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or you could do require('lodash/padStart').

Since lodash is the most depended on package in npm most have it in their npm cache.
It's essentially free.

@lydell lydell closed this in 495bb68 Mar 23, 2016
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