intersystems / ipm

InterSystems ObjectScript Package Manager
MIT License
28 stars 19 forks source link

v1 - reintroduce support for inc/rtn under PKG resource #491

Open isc-tleavitt opened 1 month ago

isc-tleavitt commented 1 month ago

Fixes #483

isc-tleavitt commented 1 month ago

@isc-kiyer @isc-bzhang I was unable to reproduce the issues related to this that were brought up in past conversation, though I did add unit tests around the case that we were worried about (that passed without needing any related code changes beyond reintroducing the line that was removed).

If you have or can find more information on why the other types were removed, that would be great. Is it possible that the source control extension could be misbehaving here?

isc-tleavitt commented 2 weeks ago

@isc-bzhang (if you have bandwidth) @isc-kiyer (if not) bumping this back up. Would be good to know what the issues really were.