Less.js: 3.8.0 '../lib/less/math-constants' -> '../lib/less/constants' rename breaks semver

[ojab]

As can be seen in https://github.com/ant-design/ant-design/issues/11417 — 3.7.1 -> 3.8.0 update breaks real world packages, so it should be named 4.0.0 and this change should be reverted for 3.x branch.

[matthew-dean]

@ojab How could this cause an issue? Internally, there are no Less references to math-constants, so I don't see why something would say the module could not be found, and the file itself hasn't existed for very long, and it's not on a public API.

[anantoghosh]

I dug into this, it's a yarn bug
https://github.com/yarnpkg/yarn/issues/6139
reported and supposedly it's already solved in the nightly build

[matthew-dean]

@anantoghosh Thanks for tracking that down! That was a confusing bug report.

[anantoghosh]

For people who are facing this issue:

• Remove less from your project
• Remove all the less packages from yarn cache
• Add the latest less again
• Verify with yarn check

[ojab]

Thanks! I've assumed that math-constants is used somewhere in antd & it's rename have caused an issue.
Sorry for the confusion.

[kristiehowboutdat]

yarn cache clean && rm -rf node_modules/ && yarn worked for me (didn't have to remove less specifically)

[danibuiza]

I also had to reinstall all packages again, probably reinstalling less would suffice