@W-21771476: feat: add HasVpi to package version list and report queries [DO NOT MERGE until 6/4 262.8 deploy]#875
Conversation
|
Thanks for the contribution! It looks like @tjporcaro is an internal user so signing the CLA is not required. However, we need to confirm this. |
4252b5a to
9f12317
Compare
|
The 5 failing tests exist on main and are caused by:
Our HasVpi tests pass:
|
|
@KSCCO this pr containing packaging library changes to support HasVpi in the version list and report queries also require your review. This is consumed by the dependent plugin changes. Thanks! |
KSCCO
left a comment
There was a problem hiding this comment.
left a couple comments, you can consider them optional
|
@shetzel These packaging lib changes should be good to merge now assuming they will be included in next Wed's cadence promotion to RC and stable the following Wed. fyi ... Once these merge I will also need to bump the dependency version on my downstream packaging plugin PR and merge those in next. See salesforcecli/plugin-packaging#1213 |
|
@iowillhoit You are correct. It turned out to be a an issue with my new tests not restoring the api version on the connection, thus affecting downstream tests . I pushed a commit and all nuts now passing now. Thanks! |
@W-21771476@
Summary
HasVpiboolean field to package version list and report queriesPackage2VersionFields,verboseFields(list), anddefault67Fields(report)Dependencies
Test plan
constructQuery(67, verbose)includes HasVpi,constructQuery(66, verbose)excludes itgetPackage2VersionFieldsat v66 excludes, v67 includes🤖 Generated with Claude Code