8000 Remove leftover print statement by bachya · Pull Request #38163 · home-assistant/core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Remove leftover print statement #38163

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 24, 2020
Merged

Remove leftover print statement #38163

merged 1 commit into from
Jul 24, 2020

Conversation

bachya
Copy link
Contributor
@bachya bachya commented Jul 24, 2020

Proposed change

This PR removes a leftover print statement.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • New feature (which adds functionality to an existing integration)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist
  • The code has been formatted using Black (black --fast homeassistant tests)
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • The manifest file has all fields filled out correctly.
    Updated and included derived files by running: python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt.
    Updated by running python3 -m script.gen_requirements_all.
  • Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

  • No score or internal
  • 🥈 Silver
  • 🥇 Gold
  • 🏆 Platinum

@frenck frenck merged commit 5f6bd22 into home-assistant:dev Jul 24, 2020
frenck pushed a commit that referenced this pull request Jul 24, 2020
@frenck frenck mentioned this pull request Jul 24, 2020
@mew1033
Copy link
mew1033 commented Jul 24, 2020

https://youtu.be/Dd7FixvoKBw?t=140

hahn-th pushed a commit to hahn-th/core that referenced this pull request May 2, 2025
* Document sensors and diagnostic sensors

* Fix list of diagnostic sensors enabled by default

* Fix capitalization of L1/L2/L3 and N

* Change sensors from table to list

* Fix special formatting and other lcleanup

* Fix coderabbit nitpick suggestions

* List available states for ambient humidity/temperature status

* Complete manual merge from 'current' into document-sensors

* Add states where defined; fix incorrect units on Input Frequency Status

* tiny tweak

---------

Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
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.

4 participants
0