search

zowe / zowe-install-packaging

Packaging repository for the Zowe install scripts and files
Eclipse Public License 2.0
27 stars 52 forks source link

Handle null to blank for configmgr update #3785

Closed 1000TurquoisePogs closed 6 months ago

1000TurquoisePogs commented 6 months ago

A bugfix was recently merged to configmgr https://github.com/zowe/zowe-common-c/pull/442 where null lines in yaml like foo: would previously be interpreted as "foo": "" in JSON for example, instead of "foo": null This was not correct interpretation of yaml, so it was fixed, particularly to mode further away from our node code! But, it caused a regression in our env code in that content that was previously foo= was now foo=null which wont work right for shell code. So, now we change those nulls to blanks to keep the same behavior as prior while still handling YAML better than before.

github-actions[bot] commented 6 months ago

build 4937 SUCCEEDED. Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8525254319

github-actions[bot] commented 6 months ago

Test workflow 4249 is started. Running install test: Convenience Pax The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.15.1-PR-3785/zowe-2.15.1-pr-3785-4937-20240402152143.pax Running on machine: zzow08 Result: SUCCESS Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8525351279