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

Skip to content

Backport: fix: order groups in summary by execution also in scenario#5069

Merged
mstoykov merged 1 commit intov1.2.xfrom
backport5059
Aug 19, 2025
Merged

Backport: fix: order groups in summary by execution also in scenario#5069
mstoykov merged 1 commit intov1.2.xfrom
backport5059

Conversation

@mstoykov
Copy link
Contributor

What?

Backport #5059

Why?

It is a bugfix we can make part of v1.2.2

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)

In #4807 the order was fixed when there were no scenarios,
but due to how the code structured this didn't work for scenarios.

Here we basically do the same change for the scenarios.
@mstoykov mstoykov added this to the v1.2.2 milestone Aug 19, 2025
@mstoykov mstoykov requested a review from a team as a code owner August 19, 2025 08:50
@mstoykov mstoykov requested review from ankur22 and codebien and removed request for a team August 19, 2025 08:50
@mstoykov mstoykov changed the title fix: order groups in summary by execution also in scenario Backport: fix: order groups in summary by execution also in scenario Aug 19, 2025
@mstoykov mstoykov changed the base branch from master to v1.2.x August 19, 2025 08:56
@mstoykov mstoykov merged commit d584d9e into v1.2.x Aug 19, 2025
37 checks passed
@mstoykov mstoykov deleted the backport5059 branch August 19, 2025 10:17
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