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

Bump karma dev-dependency from 3.1.4 to 5.0.2 #64403

Merged
merged 3 commits into from
Apr 27, 2020
Merged

Conversation

watson
Copy link
Contributor

@watson watson commented Apr 24, 2020

No description provided.

@watson watson added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.8.0 labels Apr 24, 2020
@watson watson self-assigned this Apr 24, 2020
@watson watson changed the title Bump karma dev-dependency from 3.1.4 to 4.4.1 Bump karma dev-dependency from 3.1.4 to 5.0.2 Apr 24, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@watson watson requested a review from a team April 24, 2020 16:15
@watson watson marked this pull request as ready for review April 24, 2020 17:20
@watson watson requested a review from a team as a code owner April 24, 2020 17:20
Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

Breaking changes between the two versions from https://github.com/karma-runner/karma/releases

  • Karma plugins which rely on the fact that Karma uses Bluebird promises may break as Bluebird-specific API is no longer available on Promises returned by the Karma core
  • server: Deprecated createPreprocessor removed, karma-browserify < 7 version doesn't work
  • no more testing on node 8.
  • Drop Support for Node 6, to make it possible to use async/await in karma codebase.

LGTM!

@LeeDr
Copy link

LeeDr commented Apr 25, 2020

I'm not sure what the expectations might be for kibana-qa team as a reviewer. My first instinct is to just trust the Jenkins test results and LGTM it.

Do we think this could break Kibana on some platforms? The easiest way for us to test that is after this is merged and there are snapshot builds.

I don't want to gloss over the review. Any suggestions?

@spalger
Copy link
Contributor

spalger commented Apr 25, 2020

@LeeDr Karma is a test runner, so I don't think so

@spalger spalger removed the request for review from a team April 25, 2020 18:03
@watson
Copy link
Contributor Author

watson commented Apr 27, 2020

Thanks for the review ☺️ I originally requested the review from the QA team because, as Spencer mentions, that Karma is a test runner, so I thought it felt under the QA team. Sorry for the noise.

@watson watson merged commit 3c5eee7 into elastic:master Apr 27, 2020
@watson watson deleted the bump-karma branch April 27, 2020 06:41
watson added a commit to watson/kibana that referenced this pull request Apr 27, 2020
watson added a commit to watson/kibana that referenced this pull request Apr 27, 2020
watson added a commit that referenced this pull request Apr 27, 2020
This was brought on by merging #64401, #64403, #64406, and #64433.
gmmorris added a commit to gmmorris/kibana that referenced this pull request Apr 27, 2020
* master: (78 commits)
  Remove SO root property index signature (elastic#64434)
  [ML] Functional tests - stabilize job row details validations (elastic#64503)
  [Ingest] Add Global settings flyout (elastic#64276)
  Bump cypress dev-dependency from 4.2.0 to 4.4.1 (elastic#64408)
  Migrate saved object of type url to kibana platform (elastic#64043)
  [NP] Migrate ui capabilities (elastic#64185)
  Bump karma-mocha dev-dependency from 1.3.0 to 2.0.0 (elastic#64407)
  Migrate kql_telemetry saved object registration to Kibana platform (elastic#64149)
  Remove SO autocreateindex error and error page (elastic#64037)
  Fix issue with yarn.lock (elastic#64496)
  Bump @hapi/boom dependency from 7.4.2 to 7.4.11 (elastic#64433)
  Bump gonzales-pe dev-dependency from 4.2.4 to 4.3.0 (elastic#64401)
  Bump karma dev-dependency from 3.1.4 to 5.0.2 (elastic#64403)
  Bump iedriver dev-dependency from 3.14.1 to 3.14.2 (elastic#64406)
  Revert "disable flaky suite (elastic#64382) (elastic#64381) (elastic#64480)"
  Revert "skip flaky suite (elastic#64310)"
  skip flaky suite (elastic#64310)
  disable flaky suite (elastic#64382) (elastic#64381) (elastic#64480)
  [Maps] Include maps styles in embeddables factory so that they're bundled w/ consuming plugins (elastic#64460)
  update charts library to 18.4.2 (elastic#64438)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v7.8.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants