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

docs: adding exceptions to Vale rule #15431

Merged
merged 2 commits into from
Dec 17, 2024
Merged

docs: adding exceptions to Vale rule #15431

merged 2 commits into from
Dec 17, 2024

Conversation

JStickler
Copy link
Contributor

What this PR does / why we need it:

The top two files with the most Vale spelling errors are the two configuration references. Adding exceptions to the Vale spelling rules for the:

  • Loki configuration reference file
  • Loki Helm reference file

@JStickler JStickler added the type/docs Issues related to technical documentation; the Docs Squad uses this label across many repositories label Dec 16, 2024
@JStickler JStickler requested a review from a team as a code owner December 16, 2024 23:09

This comment has been minimized.

Copy link
Contributor

github-actions bot commented Dec 16, 2024

💻 Deploy preview deleted.

This comment has been minimized.

docs/sources/shared/configuration.md Outdated Show resolved Hide resolved
@JStickler JStickler force-pushed the 2024.12.16_ValeLinting branch from e49568c to 016dfbf Compare December 17, 2024 16:09
Copy link
Contributor

Kubernetes Manifest Diff Summary

Scenario: default-single-binary-values (Added: 0, Modified: 0, Removed: 0)

Summary:

  • Added: 0

  • Modified: 0

  • Removed: 0

Added Files

No added files

Modified Files

No modified files

Removed Files

No removed files

Scenario: default-values (Added: 0, Modified: 0, Removed: 0)

Summary:

  • Added: 0

  • Modified: 0

  • Removed: 0

Added Files

No added files

Modified Files

No modified files

Removed Files

No removed files

Scenario: ingress-values (Added: 0, Modified: 0, Removed: 0)

Summary:

  • Added: 0

  • Modified: 0

  • Removed: 0

Added Files

No added files

Modified Files

No modified files

Removed Files

No removed files

Scenario: legacy-monitoring-values (Added: 0, Modified: 0, Removed: 0)

Summary:

  • Added: 0

  • Modified: 0

  • Removed: 0

Added Files

No added files

Modified Files

No modified files

Removed Files

No removed files

Scenario: simple-scalable-aws-kube-irsa-values (Added: 0, Modified: 0, Removed: 0)

Summary:

  • Added: 0

  • Modified: 0

  • Removed: 0

Added Files

No added files

Modified Files

No modified files

Removed Files

No removed files

@JStickler JStickler merged commit d75698c into main Dec 17, 2024
64 checks passed
@JStickler JStickler deleted the 2024.12.16_ValeLinting branch December 17, 2024 16:28
loki-gh-app bot pushed a commit that referenced this pull request Dec 17, 2024
loki-gh-app bot pushed a commit that referenced this pull request Dec 17, 2024
loki-gh-app bot pushed a commit that referenced this pull request Dec 17, 2024
loki-gh-app bot pushed a commit that referenced this pull request Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/helm backport release-3.0.x backport release-3.1.x backport release-3.2.x backport release-3.3.x size/M type/docs Issues related to technical documentation; the Docs Squad uses this label across many repositories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants