Skip to content

Coverage.supported? recognizes oneshot_lines mode#7040

Merged
ioquatix merged 1 commit intoruby:masterfrom
kevin-j-m:coverage-supported-oneshot
Dec 29, 2022
Merged

Coverage.supported? recognizes oneshot_lines mode#7040
ioquatix merged 1 commit intoruby:masterfrom
kevin-j-m:coverage-supported-oneshot

Conversation

@kevin-j-m
Copy link
Contributor

This adds oneshot_lines as a mode of coverage that is supported.

This adds oneshot_lines as a mode of coverage that is supported.
@kevin-j-m
Copy link
Contributor Author

Sorry, I didn't reference the issue in the commit message, as recommended.

I can force-push an edited commit, if that's alright/accepted practice.

Copy link
Member

@ioquatix ioquatix left a comment

Choose a reason for hiding this comment

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

LGTM.

@ioquatix ioquatix requested a review from mame December 29, 2022 01:01
@ioquatix ioquatix merged commit b3d330c into ruby:master Dec 29, 2022
@kevin-j-m kevin-j-m deleted the coverage-supported-oneshot branch December 29, 2022 13:06
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