Practically, the default value for the --receipts-show-limit flag that the action passes is an empty string, which does not convert to a number. In that case, it needs an extra check if it is an empty string or not, the same pattern we apply to the other flags/variables.
Use Cases
Checklist
[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).
pacostas
commented
3 months ago
Summary
This PR fixes below issue:
Practically, the default value for the
--receipts-show-limitflag that the action passes is an empty string, which does not convert to a number. In that case, it needs an extra check if it is an empty string or not, the same pattern we apply to the other flags/variables.Use Cases
Checklist