Skip to content

Conversation

klaricch
Copy link
Contributor

No description provided.

@klaricch klaricch requested a review from jkgoodrich January 25, 2022 16:18
Copy link
Contributor

@jkgoodrich jkgoodrich left a comment

Choose a reason for hiding this comment

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

Just requesting a couple loggers, and addition to docstring params. Thanks for adding!

@klaricch klaricch requested a review from jkgoodrich January 25, 2022 16:42
Copy link
Contributor

@jkgoodrich jkgoodrich left a comment

Choose a reason for hiding this comment

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

Tiny changes

@klaricch klaricch requested a review from jkgoodrich January 25, 2022 16:56
Copy link
Contributor

@jkgoodrich jkgoodrich left a comment

Choose a reason for hiding this comment

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

LGTM!

@klaricch klaricch merged commit b955780 into master Jan 25, 2022
@klaricch klaricch deleted the qc_mt branch January 25, 2022 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants