Skip to content

docs: warn that a .talon file shadows a same-named .talon-list - #470

Merged
AndreasArvidsson merged 2 commits into
TalonCommunity:mainfrom
wesleyboar:docs/talon-list-filename-shadowing
Sep 5, 2026
Merged

docs: warn that a .talon file shadows a same-named .talon-list#470
AndreasArvidsson merged 2 commits into
TalonCommunity:mainfrom
wesleyboar:docs/talon-list-filename-shadowing

Conversation

@wesleyboar

@wesleyboar wesleyboar commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Overview

Warns that a .talon file silently shadows a .talon-list of the same basename in the same directory.

Related

Changes

  • added a :::warning admonition to talon_lists.md

Testing

  1. Open .talon-list Files.
  2. Verify nothing mentions filename collisions.
  3. Build this branch.
  4. Verify a warning now appears above the user.key_special example.

Notes

  • Talon keys loaded files by path-without-extension, so only one of the pair survives. The list registers with zero entries and its captures stop matching, with no warning or error.
  • This problem is live in talonhub/community today: user.emoji and user.top_level_domain both load empty.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@netlify

netlify Bot commented Sep 3, 2026

Copy link
Copy Markdown

Deploy Preview for talon-wiki-refactor ready!

Name Link
🔨 Latest commit 67edb6f
🔍 Latest deploy log https://app.netlify.com/projects/talon-wiki-refactor/deploys/6a9c478ea23c04000880b325
😎 Deploy Preview https://deploy-preview-470--talon-wiki-refactor.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.

Comment thread docs/Customization/talon_lists.md Outdated
@AndreasArvidsson
AndreasArvidsson merged commit b03f478 into TalonCommunity:main Sep 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants