Skip to content

Conversation

misch2
Copy link
Contributor

@misch2 misch2 commented Aug 2, 2022

Summary

Tests in t/mojo/reactor_ev.t converted to use subtests.

Motivation

Help convert remaining subtests.

References

#1520

@kraih
Copy link
Member

kraih commented Aug 2, 2022

Please squash commits.

@misch2 misch2 force-pushed the michals/convert-tests-to-use-subtests-reactor-ev branch from fc2f924 to e67753f Compare August 3, 2022 12:11
@misch2
Copy link
Contributor Author

misch2 commented Aug 3, 2022

Commits squashed

marcusramberg
marcusramberg previously approved these changes Aug 14, 2022
Copy link
Member

@marcusramberg marcusramberg left a comment

Choose a reason for hiding this comment

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

LGTM

@jhthorsen
Copy link
Member

I also think you might want to squash the commits and write a more descriptive commit message.

@misch2 misch2 force-pushed the michals/convert-tests-to-use-subtests-reactor-ev branch from 6060e16 to 5a8c74d Compare August 18, 2022 16:21
@misch2
Copy link
Contributor Author

misch2 commented Aug 18, 2022

Commits squashed.

@misch2 misch2 force-pushed the michals/convert-tests-to-use-subtests-reactor-ev branch from dc9e7e4 to ed0a5e9 Compare August 22, 2022 17:40
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.

4 participants