phetsims / perennial

Maintenance tools that won't change with different versions of chipper checked out
MIT License
2 stars 5 forks source link

updateDependencies should check if working copy is clean #329

Open zepumph opened 1 year ago

zepumph commented 1 year ago

While doing an MR for https://github.com/phetsims/phet-io-wrappers/issues/519, I accidentally had some working copy changes in perennial-alias instead of perennial for maintenance code. In this case, updateDependencies really failed in a gross way. It got partially through pushing changes and the internal maintenance json state was really messed up. @jonathanolson can we please do a double check here? Similar to how we do it for dev versions:

https://github.com/phetsims/perennial/blob/2977b6222cc4c589cd3fe99fa58a513ff786ed49/js/grunt/dev.js#L71-L79