Skip to content
This repository has been archived by the owner on Jan 30, 2025. It is now read-only.

Update to latest k6 and fixes for timeseries #565

Merged
merged 3 commits into from
Oct 5, 2022
Merged

Update to latest k6 and fixes for timeseries #565

merged 3 commits into from
Oct 5, 2022

Conversation

mstoykov
Copy link
Contributor

@mstoykov mstoykov commented Oct 5, 2022

No description provided.

@mstoykov mstoykov added this to the v0.6.0 milestone Oct 5, 2022
@mstoykov mstoykov requested a review from ankur22 October 5, 2022 12:13
@mstoykov
Copy link
Contributor Author

mstoykov commented Oct 5, 2022

The 1.16.x tests were not passing for some dependency reason, so I decided to just bump the go version across the CI while I am at it.

Also see grafana/k6#2474 for a bug that only gets fixed by going to go 1.19

Copy link
Contributor

@imiric imiric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 🙇

So this mostly closes #279, and what's left would be to do the interface{} to any change, as well as test the CI release process. We still anticipate release failures caused by Go 1.18, similar to what happened in grafana/k6#2588, right?

@mstoykov
Copy link
Contributor Author

mstoykov commented Oct 5, 2022

I am porting grafana/k6#2591 to xk6-browser, but maybe I can do that in a separate PR as that will also ... update a bunch of stuff

@imiric
Copy link
Contributor

imiric commented Oct 5, 2022

@mstoykov Thanks, a separate PR would make more sense for that.

@mstoykov mstoykov merged commit 3da774f into main Oct 5, 2022
@mstoykov mstoykov deleted the updateK6 branch October 5, 2022 15:20
@inancgumus inancgumus added the dependencies Pull requests that update a dependency file label Oct 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants