Skip to content
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

Upgrade experimental module (deps): Bump github.com/grafana/xk6-browser from 1.3.0 to 1.4.1 #3625

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 8, 2024

Bumps github.com/grafana/xk6-browser from 1.3.0 to 1.4.1.

Release notes

Sourced from github.com/grafana/xk6-browser's releases.

v1.4.1

Release notes haven't been published and are in draft form here: https://github.com/grafana/k6/blob/release-v0.50.0/release%20notes/v0.50.0.md

Commits
  • a055418 Fix ElementHandle.setInputFiles
  • 8326296 Add tests for setInputFiles() in the injected script
  • cd35c13 Add tests for SetInputFiles functionality in ElementHandle
  • 027d0bf Add test for SetInputFiles function
  • b7010aa Add SetInputFiles functionality to Page and Frame
  • 273240b Refactor: separate SetInputFiles parameter from options
  • d9d2009 Add capability to set input files in multiple form
  • 38c1a6b Add setInputFiles to ElementHandle
  • 93edd08 Add testRunId to the k6 object
  • 3bbd316 Add opts to startIteration
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested a review from a team as a code owner March 8, 2024 12:42
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Mar 8, 2024
@dependabot dependabot bot requested review from mstoykov, oleiade and a team and removed request for a team March 8, 2024 12:42
mstoykov
mstoykov previously approved these changes Mar 8, 2024
go.mod Outdated
@@ -36,7 +36,7 @@ require (
github.com/spf13/afero v1.1.2
github.com/spf13/cobra v1.4.0
github.com/spf13/pflag v1.0.5
github.com/stretchr/testify v1.8.4
github.com/stretchr/testify v1.9.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you fix this @grafana/k6-browser unless you need some new functionality in it

Copy link
Member

@inancgumus inancgumus Mar 11, 2024

Choose a reason for hiding this comment

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

No, we don't need the new functionality, but if it doesn't break the k6-core functionality, is there any harm in upgrading it? It's only used for tests.

Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a strong aversion to this upgrade?

Copy link
Contributor

Choose a reason for hiding this comment

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

I do prefer that you release .1 release with reverting this and make certain you don't just bump dependencies without need in the future.

While this is not strictly necessary for the code - so is all the linting and formatting we do. I would prefer not to have to make exceptions for cases where there is no need.

@codecov-commenter
Copy link

codecov-commenter commented Mar 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.57%. Comparing base (c9dca56) to head (0a7942e).
Report is 3 commits behind head on master.

❗ Current head 0a7942e differs from pull request most recent head 0cce593. Consider uploading reports for the commit 0cce593 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3625      +/-   ##
==========================================
- Coverage   73.59%   73.57%   -0.02%     
==========================================
  Files         277      277              
  Lines       20244    20244              
==========================================
- Hits        14898    14895       -3     
- Misses       4398     4401       +3     
  Partials      948      948              
Flag Coverage Δ
macos 73.48% <ø> (?)
ubuntu 73.51% <ø> (-0.02%) ⬇️
windows 73.42% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mstoykov mstoykov added this to the v0.50.0 milestone Mar 11, 2024
@mstoykov mstoykov mentioned this pull request Mar 11, 2024
5 tasks
oleiade
oleiade previously approved these changes Mar 11, 2024
Bumps [github.com/grafana/xk6-browser](https://github.com/grafana/xk6-browser) from 1.3.0 to 1.4.0.
- [Release notes](https://github.com/grafana/xk6-browser/releases)
- [Commits](grafana/xk6-browser@v1.3.0...v1.4.0)

---
updated-dependencies:
- dependency-name: github.com/grafana/xk6-browser
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot dismissed stale reviews from oleiade and mstoykov via 934fe2b March 11, 2024 14:32
@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/grafana/xk6-browser-1.4.0 branch from 7e8b47b to 934fe2b Compare March 11, 2024 14:32
@ankur22 ankur22 changed the title Upgrade experimental module (deps): Bump github.com/grafana/xk6-browser from 1.3.0 to 1.4.0 Upgrade experimental module (deps): Bump github.com/grafana/xk6-browser from 1.3.0 to 1.4.1 Mar 11, 2024
Copy link
Contributor Author

dependabot bot commented on behalf of github Mar 11, 2024

A newer version of github.com/grafana/xk6-browser exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@mstoykov mstoykov requested a review from oleiade March 11, 2024 15:24
@mstoykov mstoykov merged commit 7cebb27 into master Mar 11, 2024
24 checks passed
@mstoykov mstoykov deleted the dependabot/go_modules/github.com/grafana/xk6-browser-1.4.0 branch March 11, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants