Skip to content

Create StylesPath prior to vale sync#142

Merged
theletterf merged 1 commit into
mainfrom
cotti-patch-1
May 16, 2026
Merged

Create StylesPath prior to vale sync#142
theletterf merged 1 commit into
mainfrom
cotti-patch-1

Conversation

@cotti
Copy link
Copy Markdown
Contributor

@cotti cotti commented May 15, 2026

This pull request ensures the .vale-styles directory exists before running vale sync, addressing a requirement introduced in Vale 3.14.2+.

More info: vale-cli/vale#1096

@cotti cotti self-assigned this May 15, 2026
@cotti cotti requested a review from theletterf as a code owner May 15, 2026 22:12
@cotti cotti added the bug Something isn't working label May 15, 2026
@theletterf theletterf merged commit 1c65891 into main May 16, 2026
8 of 9 checks passed
@theletterf
Copy link
Copy Markdown
Member

Follow-up: this change broke downstream Vale CI because mkdir -p .vale-styles was added inside the heredoc that writes .vale.ini.temp. Vale then parsed it as config and failed during vale sync with:

key-value delimiter not found: mkdir -p .vale-styles

Fixed on main by moving the .vale-styles directory creation before the heredoc so it runs as shell instead of being written into the Vale config.

Fix commit: 241c453

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants