Azure-Player / azure.datafactory.tools

Tools for deploying Data Factory (v2) in Microsoft Azure
https://azureplayer.net/adftools
MIT License
209 stars 69 forks source link

Config boolean issue #278

Closed tctylerFC closed 1 year ago

tctylerFC commented 1 year ago

Removing Dry-Run $value setting causing config validation to throw an error when Boolean variables are included.

tctylerFC commented 1 year ago

Follow-up PR based on the discussion within #273

tctylerFC commented 1 year ago

@NowinskiK could you review and approve this PR? Or is this not the change you suggested in the previous PR?

tctylerFC commented 1 year ago

So, relatively new to committing to public repos; if I'm not following good etiquette, feel free to let me know so that I can adjust my behavior. Is there something else I can provide? Would you prefer I create a bug and cancel this PR? Please let me know how I can further assist in correcting this issue. Thanks again for providing this whole toolset; looking forward to hearing back.