Skip to content

Conversation

@saravananselvamohan
Copy link
Contributor

Summary

  1. Fix Change plugin id for triggers_actions_ui to triggersActionsUi #59191
  2. Renamed triggers_actions_ui plugin id to triggersActionsUi to handle camelCase format warning
  3. Link PR's Renamed id triggersActionsUi to handle warning #77284 Rename triggers_actions_ui plugin id to triggersActionsUi #77700
  4. ✔️Verified Signature Commits

@saravananselvamohan saravananselvamohan requested review from a team as code owners September 26, 2020 14:46
@saravananselvamohan saravananselvamohan requested a review from a team September 26, 2020 14:46
@saravananselvamohan saravananselvamohan requested a review from a team as a code owner September 26, 2020 14:46
@kibanamachine
Copy link
Contributor

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@botelastic botelastic bot added Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Sep 26, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@botelastic botelastic bot added Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Sep 26, 2020
@shahzad31 shahzad31 added v7.10.0 v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels Sep 28, 2020
@shahzad31
Copy link
Contributor

Jenkins, test this

Copy link
Contributor

@shahzad31 shahzad31 left a comment

Choose a reason for hiding this comment

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

Uptime changes LGTM !!

Copy link
Contributor

@YulNaumenko YulNaumenko left a comment

Choose a reason for hiding this comment

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

LGTM

@YulNaumenko
Copy link
Contributor

jenkins retest this please

@flash1293
Copy link
Contributor

@saravananselvamohan to make the build pass you have to run the docs generation script and commit the result:

ERROR: 'node scripts/build_plugin_list_docs' caused changes to the following files:

@saravananselvamohan
Copy link
Contributor Author

@saravananselvamohan to make the build pass you have to run the docs generation script and commit the result:

ERROR: 'node scripts/build_plugin_list_docs' caused changes to the following files:

@flash1293. Thanks for the note. I checked for how to run the docs generation script in contributing guidelines. Could you brief the process, please

@flash1293 flash1293 self-assigned this Sep 30, 2020
@flash1293
Copy link
Contributor

@saravananselvamohan You just have to run node scripts/build_plugin_list_docs in your terminal. This should edit some files which you have to commit. Also, it seems like in the meantime some conflicts came in, can you check them out?

Copy link
Member

@simianhacker simianhacker left a comment

Choose a reason for hiding this comment

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

LGTM

@saravananselvamohan
Copy link
Contributor Author

@saravananselvamohan You just have to run node scripts/build_plugin_list_docs in your terminal. This should edit some files which you have to commit. Also, it seems like in the meantime some conflicts came in, can you check them out?

@flash1293. Added docs generation script

@flash1293
Copy link
Contributor

@saravananselvamohan thanks! There are also some conflicts with the current state of master branch in the elastic/kibana repository. You have to resolve them before the Pr becomes mergeable

@YulNaumenko
Copy link
Contributor

@saravananselvamohan do you need a help with resolving conflicts? If you don't mind, I can resolve it and push to your branch.

@saravananselvamohan
Copy link
Contributor Author

@saravananselvamohan do you need a help with resolving conflicts? If you don't mind, I can resolve it and push to your branch.

Yeah sure. Don't want to have changes pending for more time. Thanks for the help for the past few weeks

@YulNaumenko
Copy link
Contributor

jenkins retest this please

@YulNaumenko
Copy link
Contributor

jenkins retest this please

@YulNaumenko
Copy link
Contributor

jenkins retest this please

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
triggers_actions_ui 274 - -274
triggersActionsUi - 274 +274
total -0

async chunks size

id before after diff
apm 4.1MB 4.1MB -16.0B
infra 3.8MB 3.8MB -14.0B
monitoring 1.2MB 1.2MB -6.0B
securitySolution 10.3MB 10.3MB -22.0B
triggers_actions_ui 1.5MB - -1.5MB
triggersActionsUi - 1.5MB ⚠️ +1.5MB
uptime 1.7MB 1.7MB -24.0B
total -220.0B

miscellaneous assets size

id before after diff
triggers_actions_ui 27.7KB - -27.7KB
triggersActionsUi - 27.7KB +27.7KB
total -0.0B

page load bundle size

id before after diff
apm 46.3KB 46.3KB -4.0B
infra 178.6KB 178.6KB -12.0B
monitoring 247.7KB 247.7KB -6.0B
securitySolution 587.1KB 587.1KB -4.0B
triggers_actions_ui 148.0KB - -148.0KB
triggersActionsUi - 148.0KB +148.0KB
uptime 25.2KB 25.2KB -8.0B
total -54.0B

History

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

@YulNaumenko YulNaumenko merged commit 663175c into elastic:master Oct 5, 2020
YulNaumenko added a commit to YulNaumenko/kibana that referenced this pull request Oct 5, 2020
…lastic#78555)

* Rename triggers_actions_ui plugin id to triggersActionsUi

* Rename triggers_actions_ui plugin id to triggersActionsUi

* Removed disable linting line

* ran docs generation script

* Fixed type check issue

* FIxed eslint error

Co-authored-by: Yuliia Naumenko <yuliia.naumenko@elastic.com>
YulNaumenko added a commit that referenced this pull request Oct 5, 2020
…78555) (#79575)

* Rename triggers_actions_ui plugin id to triggersActionsUi

* Rename triggers_actions_ui plugin id to triggersActionsUi

* Removed disable linting line

* ran docs generation script

* Fixed type check issue

* FIxed eslint error

Co-authored-by: Yuliia Naumenko <yuliia.naumenko@elastic.com>

Co-authored-by: Saravanan Selvamohan <43172875+saravananselvamohan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💝community release_note:skip Skip the PR/issue when compiling release notes Team:APM - DEPRECATED Use Team:obs-ux-infra_services. Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.10.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change plugin id for triggers_actions_ui to triggersActionsUi

8 participants