Skip to content

Remove unnecessary repetition in instrumenthttp.md #2000

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tom-miseur
Copy link
Contributor

Fix unnecessary repetition

What?

A simple removal of some unnecessary repetition. LMK if this also needs to go into another repo like /next

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 run the npm start command locally and verified that the changes look good.

  • 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.

  • I have reflected my changes in the relevant folders of the two previous k6 versions of the documentation (if still applicable to previous versions).

  • I have made my changes in the docs/sources/k6/next folder of the documentation.

Related PR(s)/Issue(s)

n/a

Fix unnecessary repetition
@tom-miseur tom-miseur requested review from heitortsergent and a team as code owners July 10, 2025 21:22
@tom-miseur tom-miseur requested review from oleiade and joanlopez and removed request for a team July 10, 2025 21:22
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@heitortsergent
Copy link
Collaborator

@tom-miseur thanks for catching that! And yeah, if you could please port this to next as well, and then sign the CLA (it's reset, so it's asking all users to do it again for this repo), I can go ahead and merge this. :)

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