microsoft / devops-for-windows-apps

Sample Windows app to show devops best practices
MIT License
61 stars 27 forks source link

Removepfx #2

Closed clairernovotny closed 5 years ago

clairernovotny commented 5 years ago

This looks good, but why do we need the setCloudVariables in the version.json? it's being done with the global tool intead.

The downside of doing it in the json is that it gets set multiple times, once for each project where NB.GV is used. In this case, it's just one place, but more likely there's a library and an app, so it'd set the vars twice, etc.