NPM version emits a warning

sass-eyeglass / eyeglass-math

Math. For Sass.

Other

14 stars 5 forks source link

NPM version emits a warning #5

Open TxHawks opened 8 years ago

TxHawks commented 8 years ago

The version released on npm is quite a few commits behind master, and lacks the needs property in package.json, which was introduced in 29ad013.

strarsis commented 7 years ago

[...]
The following modules did not declare an eyeglass version:
  math
[...]

Edit: This seems to be fixed with a currently open PR: https://github.com/sass-eyeglass/eyeglass-math/pull/3/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R26 But wouldn't the needed version in that PR commit be too specific / outdated now?

strarsis commented 6 years ago

@eoneill: Could this be fixed? I just encountered the warning again during yarn install and thought that it may be nice to have this finally fixed.