Skip to content

ISAL: remove mention of Core installatioin type#41964

Merged
frenck merged 1 commit into
currentfrom
isal-remove-mention-of-core
Nov 25, 2025
Merged

ISAL: remove mention of Core installatioin type#41964
frenck merged 1 commit into
currentfrom
isal-remove-mention-of-core

Conversation

@c0ffeeca7
Copy link
Copy Markdown
Contributor

Proposed change

Intelligent Storage Acceleration integration: remove mention of Core and Supervised installation types

  • as they are no longer supported

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 10:49
@c0ffeeca7 c0ffeeca7 requested a review from bdraco as a code owner November 25, 2025 10:49
@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 b19404a
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/692589ac0a1d34000897c18b
😎 Deploy Preview https://deploy-preview-41964--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 updates the ISAL (Intelligent Storage Acceleration) integration documentation by removing references to deprecated installation types (Core and Supervised). The changes reflect that only Home Assistant Operating System and Home Assistant Container are currently supported installation types with pre-installed isal.

Key changes:

  • Removes mention of deprecated "Home Assistant Core" and "Home Assistant Supervisor" installation types
  • Updates the configuration section to only reference currently supported installation types
  • Adds context about manual configuration being needed if isal has been removed

{% term "Home Assistant Operating System" %} and {% term "Home Assistant Container" %} installations already have `isal` pre-installed, and no action is required.

If your system supports [`isal`](https://github.com/pycompression/python-isal), it can be enabled with the following configuration:
If for some reason you have removed it, and your system supports [`isal`](https://github.com/pycompression/python-isal), it can be enabled with the following configuration:
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 phrase "If for some reason you have removed it" is awkward and unclear. The original text referred to "Home Assistant Core installations" which could manually install isal. The new text doesn't clarify who the audience is for this configuration section or under what circumstances someone would need to manually enable it.

Consider revising to clarify when manual configuration is needed, for example: "For manual installations or if isal has been removed, you can enable it with the following configuration:" or "If you need to manually install isal, and your system supports it, you can enable it with the following configuration:"

Suggested change
If for some reason you have removed it, and your system supports [`isal`](https://github.com/pycompression/python-isal), it can be enabled with the following configuration:
For manual installations, or if you have removed `isal` and your system supports [`isal`](https://github.com/pycompression/python-isal), you can enable it with the following configuration:

Copilot uses AI. Check for mistakes.
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 b61e089 into current Nov 25, 2025
15 checks passed
@frenck frenck deleted the isal-remove-mention-of-core branch November 25, 2025 11:01
@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