Skip to content

Add documentation for Anki integration #40019

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

Closed
wants to merge 134 commits into from

Conversation

lfavole
Copy link

@lfavole lfavole commented Jul 15, 2025

Proposed change

This PR adds the documentation for the Anki integration.

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

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.

ecspiegel and others added 30 commits July 2, 2025 19:23
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: AlCalzone <dominic.griesel@nabucasa.com>
Co-authored-by: Martin Hjelmare <marhje52@gmail.com>
Co-authored-by: Martin Hjelmare <marhje52@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
dependabot bot and others added 15 commits July 25, 2025 20:26
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: AlCalzone <dominic.griesel@nabucasa.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
@c0ffeeca7 c0ffeeca7 added the new-integration This PR adds documentation for a new Home Assistant integration label Jul 30, 2025
@c0ffeeca7
Copy link
Contributor

It looks like this PR was created based on the wrong branch.

  • Please close this PR, switch to the next branch, pull the latest next, and create a new branch from there.

@c0ffeeca7 c0ffeeca7 marked this pull request as draft July 30, 2025 04:17
@c0ffeeca7
Copy link
Contributor

I'm drafting this PR, as changes are needed. Please undraft it, once the changes have been implemented and the PR is ready for review again.

…en a retained payload was used (#40043)

Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
@lfavole
Copy link
Author

lfavole commented Jul 30, 2025

I am adding documentation for the Anki integration. So shouldn't I point it to the next branch?

@c0ffeeca7
Copy link
Contributor

I am adding documentation for the Anki integration. So shouldn't I point it to the next branch?

Yes, your 100% correct. This should be pointed against the next branch. But it seems to be based on the current branch. So you need to start your branch on the right base - which is next.

@lfavole
Copy link
Author

lfavole commented Jul 30, 2025

But why is there written that at the top of the page? It looks like it's already pointed to the next branch.

PR header: lfavole wants to merge 70 commits into home-assistant:next from lfavole:anki-integration

@c0ffeeca7
Copy link
Contributor

yes, it is pointed to the next branch. but when you first created your branch, it probably was pointed against the current branch

@c0ffeeca7
Copy link
Contributor

c0ffeeca7 commented Jul 30, 2025

your PR contains 107 changed files, and 70 commits, that's because the branch was started on current, and then pointed against next.
image

@c0ffeeca7
Copy link
Contributor

c0ffeeca7 commented Jul 30, 2025

close this PR and start new.

try something along these lines:

git status (see where you are)
git switch next
git remote -v (to see what your remote branch is called)
git pull origin next (if your remote is called origin, yours might be called lfavole or upstream or whatever you called it)
git fetch --prune origin
git checkout -b new-anki-integration-branch (this creates your new branch based on next)
then create your new integration page on that branch
git commit -m "commit message here"
git push origin new-anki-integration-branch

@lfavole lfavole closed this by deleting the head repository Jul 30, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jul 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has-parent This PR has a parent PR in another repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.