Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass link and message through when creating reports #39

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

lencioni
Copy link
Contributor

@lencioni lencioni commented Sep 6, 2024

We noticed that these are on the comparisons but not the reports. Hoping to make the reports a little more useful by passing these through.

We noticed that these are on the comparisons but not the reports. Hoping
to make the reports a little more useful by passing these through.
@lencioni lencioni requested a review from trotzig September 6, 2024 13:20
@lencioni lencioni merged commit 0ab778b into master Sep 6, 2024
4 checks passed
@lencioni lencioni deleted the link-message branch September 6, 2024 13:35
Copy link
Contributor

@trotzig trotzig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@lencioni
Copy link
Contributor Author

I just tried this in one of our repos and it seems to have fixed the issue. See here where you can see two cypress reports, one from before this fix and one from after this fix.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants