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 focuses on the important topic of Package Constructs. These Package Constructs are the functions that are in total control of what the data that's returned to users looks like. Which includes:
Package Object Full
Package Object Short
Package Object JSON
This PR moves them out of the utils file into the dedicated models directory, as what now should be done. Additionally I've gone ahead and added new tests for these constructions, as they are never tested directly (until now), while I've also found and resolved a bug of tarball_url being incorrectly included in Package Object JSON objects.
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 focuses on the important topic of Package Constructs. These Package Constructs are the functions that are in total control of what the data that's returned to users looks like. Which includes:
This PR moves them out of the
utilsfile into the dedicatedmodelsdirectory, as what now should be done. Additionally I've gone ahead and added new tests for these constructions, as they are never tested directly (until now), while I've also found and resolved a bug oftarball_urlbeing incorrectly included inPackage Object JSONobjects.