Closed sayan-biswas closed 1 year ago
Is there a test case for
-
?
@khrm There are tests for "-" result name, but I have added a test specifically for this case where result name is "-" and the filter doesn't match anything.
This PR should have a release note - we are changing the behavior of an API response. Some might consider this a "breaking change" that warrants an ACTION REQUIRED prefix.
This PR should have a release note - we are changing the behavior of an API response. Some might consider this a "breaking change" that warrants an ACTION REQUIRED prefix.
@adambkaplan Added a release note.
This behaviour was particular to ListRecords
. Now it will align with ListResults
and ListLogs
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: adambkaplan
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Changes
Removed check from ListRecord method which searches for result name when the record lister returns empty array. This was incorrect because a "-" can be used as result name to list records across multiple results.
Fixes #560
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes