Skip to content

Add pep8-naming#29

Merged
krishnans2006 merged 1 commit into
tjcsl:masterfrom
JasonGrace2282:naming
May 26, 2024
Merged

Add pep8-naming#29
krishnans2006 merged 1 commit into
tjcsl:masterfrom
JasonGrace2282:naming

Conversation

@JasonGrace2282
Copy link
Copy Markdown
Member

Tin is named decently well!

@JasonGrace2282 JasonGrace2282 requested a review from a team as a code owner May 22, 2024 18:14
Copy link
Copy Markdown
Member

@krishnans2006 krishnans2006 left a comment

Choose a reason for hiding this comment

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

LGTM!

@JasonGrace2282
Copy link
Copy Markdown
Member Author

This is also instantly compatible with #28 after switching from unittest (with camelCase conventions) to pytest!

@krishnans2006 krishnans2006 merged commit ba21058 into tjcsl:master May 26, 2024
@JasonGrace2282 JasonGrace2282 deleted the naming branch May 26, 2024 19:23
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