Skip to content
This repository was archived by the owner on Aug 29, 2024. It is now read-only.

Test: make descriptions more confident #73

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Test: make descriptions more confident #73

merged 1 commit into from
Nov 10, 2023

Conversation

olleolleolle
Copy link
Contributor

@olleolleolle olleolleolle commented Nov 9, 2023 •
edited
Loading

Types of Changes

This PR tries to make the names of the tests uniform. (No use of "should".)

A follow-up to the grand #72.

  • Maintenance.

Contribution

@olleolleolle olleolleolle changed the title Test: make desriptions more confident Test: make descriptions more confident Nov 9, 2023
ioquatix
ioquatix approved these changes Nov 10, 2023
View reviewed changes
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

@olleolleolle olleolleolle merged commit 7988b0c into socketry:main Nov 10, 2023
@olleolleolle olleolleolle deleted the clarify-test-descriptions branch November 10, 2023 12:48
@ioquatix
Copy link
Member

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ioquatix ioquatix ioquatix approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@olleolleolle @ioquatix