Skip to content

Conversation

@nourmat
Copy link

@nourmat nourmat commented Feb 9, 2025

This PR adds line_number attribute to failure property same as with JSON formatter: https://github.com/rspec/rspec-core/blob/main/lib/rspec/core/formatters/json_formatter.rb#L95

This is helpful to know which test fails and in what line so you can point to it easily for example using "githublink.com/org/repo/file_name#L{line_number}"

After

image

@sj26
Copy link
Owner

sj26 commented Feb 9, 2025

This is a nice idea, but is not part of the junit standard, so breaks some consumers and cannot be included sorry.

@sj26 sj26 closed this Feb 9, 2025
@nourmat
Copy link
Author

nourmat commented Feb 11, 2025

No worries, thanks for looking into it.

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