tap-github icon indicating copy to clipboard operation
tap-github copied to clipboard

Re-add `files` and `stats` to `PrCommits` stream

Open dlouseiro opened this issue 1 year ago • 0 comments

While testing out this tap, I noticed that, similarly to what happens in other variants of the tap-github out there, the Commits stream does not have the files and stats properties, as these ones can only be fetched commit by commit , but not when listing commits. These properties were once available in the Commits stream schema, but it became confusing for users that selected the stream to not see these data points materialised in their target (creating issues like this), hence they were removed by @dsprayberry in this PR.

In this same PR, the files and stats properties were also removed from the PrCommits stream (as they are also commit related streams). Although, unlike the Commits stream, these properties are available while listing commits for a PR, hence could be available in the PrCommits stream, even if they're not available in the Commits stream.

Functionally speaking, I'd also prefer that Github would keep their Commit related endpoints aligned and make files and stats available in both. Unfortunately this is not the case, but I still see the analytical value of having these fields available, even if only for commits that are linked to a Pull Request.

dlouseiro avatar Jan 29 '24 11:01 dlouseiro