Skip to content

Removed dead code#42774

Merged
igoristic merged 6 commits intoelastic:masterfrom
igoristic:remove-dead-code
Aug 12, 2019
Merged

Removed dead code#42774
igoristic merged 6 commits intoelastic:masterfrom
igoristic:remove-dead-code

Conversation

@igoristic
Copy link
Contributor

Removed the last of KuiTable and some other left over code

@igoristic igoristic added enhancement New value added to drive a business result Team:Monitoring Stack Monitoring team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.4.0 v7.3.1 labels Aug 6, 2019
@igoristic igoristic requested a review from a team as a code owner August 6, 2019 21:23
@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring

Copy link
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

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

Excellent. Thank you!

@elasticmachine
Copy link
Contributor

💔 Build Failed

Copy link
Contributor

@chrisronline chrisronline left a comment

Choose a reason for hiding this comment

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

LGTM, once tests pass!

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@igoristic igoristic merged commit ce5d7fb into elastic:master Aug 12, 2019
@igoristic igoristic deleted the remove-dead-code branch August 12, 2019 22:21
igoristic added a commit to igoristic/kibana that referenced this pull request Aug 12, 2019
* Removed dead code

* Fixed i18n messages

* Removed unused
igoristic added a commit to igoristic/kibana that referenced this pull request Aug 12, 2019
* Removed dead code

* Fixed i18n messages

* Removed unused
igoristic added a commit that referenced this pull request Aug 13, 2019
* Removed dead code

* Fixed i18n messages

* Removed unused
igoristic added a commit that referenced this pull request Aug 13, 2019
* Removed dead code

* Fixed i18n messages

* Removed unused
@igoristic
Copy link
Contributor Author

Backport:
7.x: 6c88619
7.3: a6de559

@cachedout
Copy link
Contributor

@igoristic and @chrisronline Does this address all the remaining work for @elastic/stack-monitoring that's shown in this spreadsheet or is there more to be done?

jloleysens added a commit to jloleysens/kibana that referenced this pull request Aug 13, 2019
…p-metrics-selectall

* 'master' of github.com:elastic/kibana: (27 commits)
  [ML] Data Frames: Analytics job creation. (elastic#43102)
  [Vis Default editor] Fix issue with Rollup (elastic#42430)
  [Vis: Default editor] EUIficate Markdown tab (elastic#42677)
  [New Platform Migration Phase I]: update dateHisogramInterval & parseEsInterval imports (elastic#42917)
  [Infra UI] Add AWS metrics to node detail page (elastic#42153)
  update apm index pattern (elastic#43106)
  [SIEM] Toggle Column / Code Coverage and Cypress (elastic#42766)
  skip failing test (elastic#43163)
  [code] Add option to turn the go dependency download on/off. (elastic#43096)
  disable visual regression jobs
  Removed dead code (elastic#42774)
  fixes csv export of saved searches that have _source field (elastic#43123)
  Export missing Context types (elastic#43051)
  Update dependency supports-color to v7 (elastic#43064)
  switch to icon type string instead of node (elastic#43111)
  [Maps] Enable borders for icon symbols (elastic#43066)
  [ftr] enable visualRegression jobs (elastic#42989)
  [ML] Converting single to multi metric job (elastic#42532)
  fix(NA): dont clean dll module if it is a package json file (elastic#42904)
  [Logs UI] Add link from the sample web logs to the Logs UI (elastic#42635)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New value added to drive a business result release_note:skip Skip the PR/issue when compiling release notes Team:Monitoring Stack Monitoring team v7.3.1 v7.4.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants