Skip to content

ci: exclude code and file trees from textlint - #14782

Merged
delucis merged 2 commits into
withastro:mainfrom
Yusaku01:feat/ja-textlint-allowlist
Oct 26, 2026
Merged

delucis merged 2 commits into
withastro:mainfrom
Yusaku01:feat/ja-textlint-allowlist

Conversation

@Yusaku01

@Yusaku01 Yusaku01 commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Description (required)

Japanese textlint reports spacing errors inside HTML <code> elements and <FileTree> content. This can require translators to change source markup or remove spaces that separate filenames from comments.

Add shared allowlist patterns for these ranges. Existing spacing rules still apply outside them; all content inside them, including FileTree comments, is excluded. Documentation fixes, including the installation guide's FileTree spacing, will be handled separately.

Validated with pnpm run lint:ja and eight before/after lint fixtures, including checks that violations in surrounding prose are still reported.

References

@github-actions github-actions Bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Oct 25, 2026
@Yusaku01 Yusaku01 changed the title ci(ja): exclude code and file trees from textlint [i18nIgnore] ci(ja): exclude code and file trees from textlint Oct 25, 2026
@github-actions

github-actions Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Preview deployment

✅ Deployment complete!

@Yusaku01
Yusaku01 marked this pull request as ready for review October 25, 2026 15:42
@Yusaku01 Yusaku01 changed the title ci(ja): exclude code and file trees from textlint ci: exclude code and file trees from textlint Oct 25, 2026

@delucis delucis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the fix @Yusaku01 🙌

@delucis
delucis merged commit cdb323d into withastro:main Oct 26, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants