Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

Fix loading files with spaces - fixes #5437#5439

Merged
mstoykov merged 1 commit intomasterfrom
testWithSpace
Nov 21, 2025
Merged

Fix loading files with spaces - fixes #5437#5439
mstoykov merged 1 commit intomasterfrom
testWithSpace

Conversation

@mstoykov
Copy link
Contributor

What?

Fixes test that have spaces in their path

Why?

This should be working, and I am extremely suprised that we do not have a test and it took over a week to get caught.

Checklist

  • I have performed a self-review of my code.
  • I have commented on my code, particularly in hard-to-understand areas.
  • I have added tests for my changes.
  • I have run linter and tests locally (make check) and all pass.

Checklist: Documentation (only for k6 maintainers and if relevant)

Please do not merge this PR until the following items are filled out.

  • I have added the correct milestone and labels to the PR.
  • I have updated the release notes: link
  • I have updated or added an issue to the k6-documentation: grafana/k6-docs#NUMBER if applicable
  • I have updated or added an issue to the TypeScript definitions: grafana/k6-DefinitelyTyped#NUMBER if applicable

Related PR(s)/Issue(s)

@mstoykov mstoykov added this to the v1.5.0 milestone Nov 20, 2025
@mstoykov mstoykov requested a review from a team as a code owner November 20, 2025 22:14
@mstoykov mstoykov requested review from codebien and oleiade and removed request for a team November 20, 2025 22:14
@mstoykov mstoykov temporarily deployed to azure-trusted-signing November 20, 2025 22:20 — with GitHub Actions Inactive
@mstoykov mstoykov temporarily deployed to azure-trusted-signing November 20, 2025 22:21 — with GitHub Actions Inactive
@mstoykov mstoykov merged commit 617faf7 into master Nov 21, 2025
41 checks passed
@mstoykov mstoykov deleted the testWithSpace branch November 21, 2025 14:31
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