Skip to content

Remove non-existent vale command in style guide - #219

Open
reniejoshi wants to merge 1 commit into
frcsoftware:mainfrom
reniejoshi:fix-style-guide-tells-you-to-run-a-non-existant-command
Open

Remove non-existent vale command in style guide#219
reniejoshi wants to merge 1 commit into
frcsoftware:mainfrom
reniejoshi:fix-style-guide-tells-you-to-run-a-non-existant-command

Conversation

@reniejoshi

Copy link
Copy Markdown
Contributor

Description

Previously, the style guide instructed contributors to run pnpm lint:vale, which is a non-existent command covered by pnpm lint. This pr removes that command for the style guide table.

Closes #215.

Meta

Merge checklist:

@github-actions github-actions Bot added the Curriculum material Curriculum materials, lessons for students, etc label Aug 31, 2026
@github-actions

Copy link
Copy Markdown

🌐 Preview URL: https://pr-219.frcsoftware.pages.dev

@httphypixelnet

Copy link
Copy Markdown
Contributor

Could we just create a new lint:vale command to replace the current vale command? Imo running pnpm vale should execute vale itself, not our script.

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

Labels

Curriculum material Curriculum materials, lessons for students, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

style guide tells you to run a non exsitant command

3 participants