zio / zio-protoquill

Quill for Scala 3
Apache License 2.0
204 stars 48 forks source link

Disable macro log do not print dynamic query detected #473

Open patextreme opened 2 months ago

patextreme commented 2 months ago

Fixes #470

Problem

-Dquill.macro.log=false doesn't silence the Dynamic Query Detected ... which become quite verbose when the project start to grow.

Solution

Add the same condition to check the flag before printing the log above so it has parity with static query logs.

Notes

Additional notes.

Checklist

@getquill/maintainers

patextreme commented 2 months ago

Resolves https://github.com/zio/zio-protoquill/issues/470

FabioPinheiro commented 2 days ago

Can we push this fix to be merge and release