require c++17

[erics118]

clang 16 may bug out on macos if it is not explicitly provided

added a line to require c++17 before linking std::filesystem

Code change checklist

• I have ensured that all methods and functions are fully documented using doxygen style comments.
• My code follows the coding style guide.
• I tested that my change works before raising the PR.
• I have ensured that I did not break any existing API calls.
• I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

[netlify]

✅ Deploy Preview for dpp-dev ready!

Name	Link
🔨 Latest commit	30a5ea7
🔍 Latest deploy log	https://app.netlify.com/sites/dpp-dev/deploys/67ea58d0199cea00087d4aa5
😎 Deploy Preview	https://deploy-preview-1404--dpp-dev.netlify.app
📱 Preview on mobile	Toggle 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]

Just add it at the top directly after project IMO

[braindigitalis]

Just add it at the top directly after project IMO

it shouldnt be hard coded to C++17. Our default is C++20 now unless its not supported or DPP_NO_CORO is set. The value for the standard should reflect this setting.

[braindigitalis]

we need to use the standard which is set by the detections not hard code to C++17, this could cause conflicts later down the line where the order of these definitions is assumed.

[Mishura4]

Just add it at the top directly after project IMO

it shouldnt be hard coded to C++17. Our default is C++20 now unless its not supported or DPP_NO_CORO is set. The value for the standard should reflect this setting.

It's not, if you set it to 17 first and then to 20, it'll be 20

i moved the declaration to the top