Skip to content

Fix minor typos in xk6-disruptor docs #1947

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

Merged
merged 1 commit into from
May 13, 2025
Merged

Conversation

al3xandrugroza
Copy link
Contributor

@al3xandrugroza al3xandrugroza commented May 7, 2025

This pull request corrects two minor typos in the xk6-disruptor documentation.

What?

Fix typo: Lear -> Learn.

In the following sentence only one period is needed.
Refer to [Expose your application](https://grafana.com/docs/k6/latest/testing-guides/injecting-faults-with-xk6-disruptor/expose--your-application/) section for instructions on how to make your application available from outside the Kubernetes cluster..

Checklist

  • I have used a meaningful title for the PR.
  • I have described the changes I've made in the "What?" section above.
  • I have performed a self-review of my changes.
  • I have made my changes in the docs/sources/k6/next folder of the documentation.
  • I have reflected my changes in the docs/sources/k6/v{most_recent_release} folder of the documentation.

@al3xandrugroza al3xandrugroza requested review from heitortsergent and a team as code owners May 7, 2025 07:03
@al3xandrugroza al3xandrugroza requested review from mstoykov and ankur22 and removed request for a team May 7, 2025 07:03
@al3xandrugroza al3xandrugroza changed the title Fix typo in xk6-disruptor docs index Fix minor typos in xk6-disruptor docs May 7, 2025
@heitortsergent
Copy link
Collaborator

@jdbaldry @robbymilo could you please help take a look and see why zizmor is failing on this PR? 🤔

@jdbaldry
Copy link
Member

I think it's known to have some trouble on forks but the security team are working on it.

I think you can ignore it in this case.

@jdbaldry jdbaldry merged commit d71b9ea into grafana:main May 13, 2025
4 of 5 checks passed
@heitortsergent
Copy link
Collaborator

Ok, thanks @jdbaldry! 🙇

And thanks @al3xandrugroza for opening the PR and contributing to the k6 docs! 🙇

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.

3 participants