You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
GitHub's PR UI shows the diff of the base branch with a ref in which the base branch has been merged into the head branch.
Locust summaries take the diff of the base branch with the head branch directly.
When the base branch contains changes not in the head branch, the Locust summary shows different modifications than the GitHub diff.
When the Locust summary is generated for GitHub, Locust should:
Try to merge the base branch into the head branch. If this succeeds, set base=base, head=base->head
If this merge fails, fall back to base=base, head=head
Take diff of base with head.
Run Locust summary generator, but replace all links to files with links to diffs (i.e. PR comments).
This last change should probably get its own issue, as it's a substantial improvement in the user experience, and seems like a significant chunk of work.
The text was updated successfully, but these errors were encountered:
GitHub's PR UI shows the diff of the base branch with a ref in which the base branch has been merged into the head branch.
Locust summaries take the diff of the base branch with the head branch directly.
When the base branch contains changes not in the head branch, the Locust summary shows different modifications than the GitHub diff.
When the Locust summary is generated for GitHub, Locust should:
base=base
,head=base->head
base=base
,head=head
base
withhead
.This last change should probably get its own issue, as it's a substantial improvement in the user experience, and seems like a significant chunk of work.
The text was updated successfully, but these errors were encountered: