chore: added a CI step to detect unused dependencies

[divinetettey]

adding this PR to resolve this issue Implement a CI step to detect unused dependencies 📦

The changes include:

1. installing the depcheck package
2. Running depcheck --oneline --skip-missing
3. Fail if the the check fails

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: JamieSlome / name: Jamie Slome (aa50a07, 4d54e57, 969b2c8, 83afd38)
• ✅ login: divinetettey / name: Divine Sedem Tettey (01f3888, 9af5654, 328c505, 3eee49d, d4110dc, b9cf2f5, 9832264)

[netlify]

✅ Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name	Link
🔨 Latest commit	d4110dc
🔍 Latest deploy log	https://app.netlify.com/sites/endearing-brigadeiros-63f9d0/deploys/6671832eace2b600082ae6be

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.39%. Comparing base (0ba98b2) to head (d4110dc).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #592   +/-   ##
=======================================
  Coverage   57.39%   57.39%           
=======================================
  Files          46       46           
  Lines        1582     1582           
=======================================
  Hits          908      908           
  Misses        674      674           

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

[JamieSlome]

@divinetettey - is there any chance we can move the check to its own workflow file? What do you think?

[divinetettey]

@divinetettey - is there any chance we can move the check to its own workflow file? What do you think?

I think that's fine, we can go with that approach

[JamieSlome]

@divinetettey - thanks for making the adjustments (perfect 🎉 ). It seems like the depcheck is reporting that concurrently and a few other devDependencies are unused which is not the case. Is there any way to add these to an allow list or as permissible exceptions?

[divinetettey]

@divinetettey - thanks for making the adjustments (perfect 🎉 ). It seems like the depcheck is reporting that concurrently and a few other devDependencies are unused which is not the case. Is there any way to add these to an allow list or as permissible exceptions?

I have added them to the ignore list. I would take a deep dive into how depcheck performs the checks

[JamieSlome]

@maoo - can we make unused dependencies required on status checks to main please?

[JamieSlome]

LGTM! 🍰

[maoo]

@maoo - can we make unused dependencies required on status checks to main please?

Done!