Skip to content

chore(pre-commit): align vale-sync pin with vale hook version - #14798

Merged
DonLakeFlyer merged 1 commit into
mavlink:masterfrom
DonLakeFlyer:vale-pin-align
Aug 9, 2026
Merged

chore(pre-commit): align vale-sync pin with vale hook version#14798
DonLakeFlyer merged 1 commit into
mavlink:masterfrom
DonLakeFlyer:vale-pin-align

Conversation

@DonLakeFlyer

Copy link
Copy Markdown
Contributor

Follow-up to #14753.

vale-sync installed github.com/errata-ai/vale/v3/cmd/vale@v3.14.2 while the vale lint hook uses repo rev v3.15.1, so styles were synced with a different vale version than the one that lints. This bumps the vale-sync dependency to v3.15.1 to keep the two pins in lockstep.

Verified locally: hook environments rebuilt from scratch (pre-commit clean), vale-sync passes with the new pin.

vale-sync installed vale@v3.14.2 while the vale lint hook uses
v3.15.1, so styles were synced with a different vale version than
the one that lints. Bump the vale-sync dependency to v3.15.1 to
keep the two in lockstep.

Follow-up to mavlink#14753.
Copilot AI balanced review requested due to automatic review settings August 9, 2026 15:52
@DonLakeFlyer DonLakeFlyer added this to the Release V5.1 milestone Aug 9, 2026
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

⚠️ Build results unavailable — artifact download from one or more platform workflows failed (likely artifact retention expiry or transient API error). The combined report cannot be generated for this run.

See the Build Results workflow run for details.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Aligns the vale-sync dependency with the Vale lint hook version to ensure consistent style synchronization and linting.

Changes:

  • Updates vale-sync from v3.14.2 to v3.15.1.
  • Keeps both Vale version pins in lockstep.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@DonLakeFlyer
DonLakeFlyer merged commit 14448a3 into mavlink:master Aug 9, 2026
29 checks passed
@DonLakeFlyer
DonLakeFlyer deleted the vale-pin-align branch August 9, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants