Chore: Add lerna as peer dependency

[jackw]

What Changed

Added lerna as a peerDependency to auto and auto/npm as they both internally call lerna with npx.

Why

Calling lerna with npx without a local copy installed could cause bugs as it will either pull the latest version from the npm registry or use a locally cached version. By setting peer dependencies lerna will either be installed when auto is installed or show a warning telling the user that lerna is a missing dependency.

@hipstersmoothie not 💯 sure about the version of lerna to support here. Do we want to widen the semver range for something like >6.0.0 <8.0.0? I'm also unsure which type of change this should be.

Todo:

• Add tests
• Add docs

Change Type

Indicate the type of change your pull request is:

• documentation
• patch
• minor
• major

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.56%. Comparing base (f6af66b) to head (7f3c773).
Report is 46 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2441      +/-   ##
==========================================
- Coverage   80.63%   80.56%   -0.08%     
==========================================
  Files          69       69              
  Lines        5680     5680              
  Branches     1277     1277              
==========================================
- Hits         4580     4576       -4     
- Misses        718      719       +1     
- Partials      382      385       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hipstersmoothie]

I think we should refactor a little bit so that all users of the all-contributors plugin don''t need to install lerna

In both places it could be an optionalPeerDep probably

[hipstersmoothie]

Hmmm it probably doesn't make sense for core to depend on lerna...

[hipstersmoothie]

only the all contributors plugin depends on that.

I think we should either:

1. move that function into a dep
2. just copy the function into the 2 places we use it (npm and all-contributors)

I think 2 is better. then we can say lerna is an optionalPeerDep of the all-contributors plugin

[jackw]

Sorry for not getting back to this sooner, I've not forgotten about it got moved down the priority list I'm afraid. I'll try and make the refactors as per your feedback asap @hipstersmoothie. 👍

[hipstersmoothie]

I completely understand :D