Skip to content

Conversation

naveen521kk
Copy link
Member

Changelog / Overview

Motivation

Removes stray print statements from #1404

Testing Status

Further Comments

Checklist

  • I have read the Contributing Guidelines
  • I have written a descriptive PR title (see top of PR template for examples)
  • I have written a changelog entry for the PR or deem it unnecessary
  • My new functions/classes either have a docstring or are private
  • My new functions/classes have tests added and (optional) examples in the docs
  • My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

  • The PR title is descriptive enough
  • The PR is labeled correctly
  • The changelog entry is completed if necessary
  • Newly added functions/classes either have a docstring or are private
  • Newly added functions/classes have tests added and (optional) examples in the docs
  • Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

@behackl behackl added infrastructure Anything related to our infrastructure maintenance refactoring, typos, removing clutter/dead code, and other code quality improvements and removed infrastructure Anything related to our infrastructure labels May 11, 2021
@behackl behackl merged commit e231eb2 into master May 11, 2021
@behackl behackl deleted the stray-prints branch May 11, 2021 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance refactoring, typos, removing clutter/dead code, and other code quality improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants