brainboxdotcc / DPP

C++ Discord API Bot Library - D++ is Lightweight and scalable for small and huge bots!
https://dpp.dev/
Apache License 2.0
1.04k stars 158 forks source link

capturing "bot" is not needed. Removing it resolves an unused variabl… #1132

Closed Erarnitox closed 4 months ago

Erarnitox commented 4 months ago

Alternatively you could please mark the bot variable as unused by casting it to void. This would clean up a warning for library consumers who happen to build unit tests also.

CLAassistant commented 4 months ago

CLA assistant check
All committers have signed the CLA.

netlify[bot] commented 4 months ago

Deploy Preview for dpp-dev ready!

Name Link
Latest commit f635c9d09cffe2c07bf4872ede7cb79ee8d24aae
Latest deploy log https://app.netlify.com/sites/dpp-dev/deploys/661ee8c6b0ba920008c9b186
Deploy Preview https://deploy-preview-1132--dpp-dev.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Mishura4 commented 4 months ago

I force rebased the PR to the latest version of dev. Waiting on the actions to complete then I'll merge