All new code requires tests to ensure against regressions.
However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
This PR extends #241 by now using this new callStack just about everywhere.
Although with #240 still in review, I made sure not to touch getPackagesPackageName and postPackages as those callStack signatures change through that PR, and I didn't want to create any conflicts.
But in short, all this PR does it make it easier to trace errors occurring on the backend, and for any reported errors, at a glance we can quickly see and read the path that request took through the code, and what failed.
Requirements
Filling out the template is required.
All new code requires tests to ensure against regressions.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
This PR extends #241 by now using this new
callStack
just about everywhere.Although with #240 still in review, I made sure not to touch
getPackagesPackageName
andpostPackages
as those callStack signatures change through that PR, and I didn't want to create any conflicts.But in short, all this PR does it make it easier to trace errors occurring on the backend, and for any reported errors, at a glance we can quickly see and read the path that request took through the code, and what failed.