Closed dalehenrich closed 1 year ago
Attention: 54 lines
in your changes are missing coverage. Please review.
Comparison is base (
80af760
) 48.67% compared to head (accfd4b
) 48.64%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The older version of Metacello being used in extent0.seaside.dbf was not properly dealing with the fact that the required packages for the newish HotwireTurbo packages were not complete and resulted in undefined globals. Adjusting the required packages for the HotwireTurbo packages fixed the package load order issue and reqolved the undefined global references.
Another point is that the Seaside-HotwireTurbo and Seaside-HotwireTurbo-Examples share a common base name for the package which is in violation of the Monticello package naming conventionsand would lead to unwanted package mangling if the HotwireTurbo were modified and saved from an image that follows the Monticello package naming convention. So I renamed Seaside-HotwireTurbo to Seaside-HotwireTurbo-Core to avoid this potential issue.