Closed agateau-gg closed 9 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
a976b98
) 91.90% compared to head (649bbf2
) 91.97%.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This PR finishes removing the
Commit.files
property, which is currently responsible for loading whole commits in memory.Changes in a Commit are no longer represented as StringScannable instances. Instead there is a dedicated class, CommitScannable. Using a dedicated class gives more control over the
path
andurl
properties.Finally, I added some test to validate the JSON output because my changes in CommitInformation initially introduced new fields in the
extra_info
object. This was because all of CommitInformation was being turned into a dict and put inextra_info
. Now we only store selected fields.Best reviewed commit by commit.