Skip to content

Logger: remove mention of Supervised#41961

Merged
frenck merged 2 commits into
currentfrom
viewing-logs-remove-supervised
Nov 25, 2025
Merged

Logger: remove mention of Supervised#41961
frenck merged 2 commits into
currentfrom
viewing-logs-remove-supervised

Conversation

@c0ffeeca7
Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 commented Nov 25, 2025

Proposed change

Logger: remove mention of Supervised

  • as it is no longer a recommended installation option
  • Fix typo
  • Tiny style tweaks

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Copilot AI review requested due to automatic review settings November 25, 2025 09:51
@c0ffeeca7 c0ffeeca7 requested a review from a team as a code owner November 25, 2025 09:51
@home-assistant home-assistant Bot added the current This PR goes into the current branch label Nov 25, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 25, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit f3a6100
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/69257c846188ed0008705f11
😎 Deploy Preview https://deploy-preview-41961--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes references to Supervised installations from the logger integration documentation, aligning with the fact that Supervised is no longer a recommended installation option. The changes simplify the "Viewing logs" section by removing outdated installation-specific instructions.

Key changes:

  • Removed the "Supervised installations" subsection that included SSH add-on instructions for Supervisor-based installations
  • Removed the "Non-Supervised installations" subsection header
  • Consolidated remaining content under a new "Viewing logs on Container installations" heading


The log information can be viewed and downloaded from {% my logs title="**Settings** > **System** > **Logs**" %}

### Viewing logs on Container installations
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

The section heading uses "Viewing logs on Container installations" but this is redundant and inconsistent with the parent section "Viewing logs" (line 172). According to the documentation standards, this creates an awkward flow.

Consider simplifying to "### Container installations" to match the style of the removed sections ("Supervised installations" and "Non-Supervised installations"). This maintains parallelism and follows the established pattern in the document.

Copilot generated this review using guidance from repository custom instructions.
Comment on lines +178 to 181
For {% term "Home Assistant Container" %} installations, the log information is stored in the
[configuration directory](/docs/configuration/) as `home-assistant.log`.
You can read it with the command-line tool `cat` or follow it dynamically
with `tail -f`.
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

The sentence structure could be improved for better readability. The current phrasing "For {% term "Home Assistant Container" %} installations the log information is stored..." is missing a comma after the introductory phrase.

According to documentation standards, this should read: "For {% term "Home Assistant Container" %} installations, the log information is stored in the configuration directory as home-assistant.log and you can read it with the command-line tool cat or follow it dynamically with tail -f."

Additionally, consider breaking this into two sentences for better clarity:
"For {% term "Home Assistant Container" %} installations, the log information is stored in the configuration directory as home-assistant.log. You can read it with the command-line tool cat or follow it dynamically with tail -f."

Copilot generated this review using guidance from repository custom instructions.
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Thanks, @c0ffeeca7 👍

../Frenck

                       

Blogging my personal ramblings at frenck.dev

@frenck frenck merged commit 96151a4 into current Nov 25, 2025
9 checks passed
@frenck frenck deleted the viewing-logs-remove-supervised branch November 25, 2025 10:02
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 26, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants