ipm
ipm copied to clipboard
v0.7.x to v0.9.x upgrade path and storage migration
Fixes #454
Upgrade Path zpm 0.7.1 (once released) will be able to install zpm 0.9.0 (once released).
Storage Migration
Moves ^ZPM.Dev.ModuleD to ^IPM.Storage.ModuleD
In ^IPM.Storage.ModuleD(<ID>,"Defaults",n) maps classname to equivalent - %ZPM.PackageManager.Developer.ModuleSetting -> %IPM.Storage.ModuleSetting
Migrates %ZPM.PackageManager.Developer.Document.ModuleStream to %IPM.StudioDocument.ModuleStream
For custom lifecycle classes, we're less picky and allow the legacy naming too. (I'm probably the only one using one of these anyway, in https://github.com/intersystems/isc-ipm-js )
Also incorporates the final CI fixes for #451.
Leaving this as a draft until CI is all passing. (I don't have time to run it all locally in act today, so let's see what happens!)
Well, that was quick. Also trying to fix the last lingering task in #451 here and that's where the CI issue is currently.
This looks like it's working. Note, it builds on #457 so review that one first.
@timleavitt a few comments. a few questions. I don't entirely understand the GitHub workflow syntax so haven't reviewed that. Would be good to get a second set of eyes on that bit
@isc-shuliu I'd appreciate if you could review the GitHub workflow parts of this, since you've worked with it more than anyone else reviewing. (And probably more than I have!)
This is ready for review (again).
Bumping this back up - needs re-review.
@isc-kiyer @isc-bzhang @isc-eneil @isc-shuliu
Need to update integration tests to interoperability-enable the VERIFY namespace.
Once again, I think we're good here - @isc-kiyer @isc-shuliu could you take a hopefully-final look?
Looks all good to me!
@shuheng-liu FYI, you switched profiles ;)