Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Vale configuration example #951

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Update Vale configuration example #951

wants to merge 2 commits into from

Conversation

jdbaldry
Copy link
Member

I need to also update the steps to tell a user to run vale sync

  • I've used a relevant pull request (PR) title.
  • I've added a link to any relevant issues in the PR description.
  • I've checked my changes on the deploy preview and they look good.
  • I've added an entry to the What's new page (only required for notable changes).

I need to also update the steps to tell a user to run `vale sync`
Copy link
Contributor

BasedOnStyles = Grafana
TokenIgnores = (<http[^\n]+>+?), \*\*[^\n]+\*\*
```

Replace _`<PATH TO WRITERS TOOLKIT REPOSITORY>`_ with the full path to your checkout of the Writer's Toolkit repository.
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
Replace _`<PATH TO WRITERS TOOLKIT REPOSITORY>`_ with the full path to your checkout of the Writer's Toolkit repository.
Replace _`<PATH TO WRITERS TOOLKIT REPOSITORY>`_ with the full path to your checkout of the Writers' Toolkit repository.

@clayton-cornell
Copy link
Contributor

clayton-cornell commented Jan 14, 2025

I had to do a <PATH>/vale --config='<PATH>/vale.ini sync to get vale working with this updated vale.ini. (I included because vale binary and vale.ini are not in my system path :-P )

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