-
Notifications
You must be signed in to change notification settings - Fork 8.5k
Change plugin id for triggers_actions_ui to triggersActionsUi
#78555
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
Change plugin id for triggers_actions_ui to triggersActionsUi
#78555
Conversation
|
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? |
|
Pinging @elastic/uptime (Team:uptime) |
|
Pinging @elastic/apm-ui (Team:apm) |
|
Jenkins, test this |
shahzad31
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uptime changes LGTM !!
YulNaumenko
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
jenkins retest this please |
|
@saravananselvamohan to make the build pass you have to run the docs generation script and commit the result:
|
@flash1293. Thanks for the note. I checked for how to run the docs generation script in contributing guidelines. Could you brief the process, please |
|
@saravananselvamohan You just have to run |
simianhacker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@flash1293. Added docs generation script |
|
@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 |
|
@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 |
|
jenkins retest this please |
|
jenkins retest this please |
|
jenkins retest this please |
💚 Build SucceededMetrics [docs]@kbn/optimizer bundle module count
async chunks size
miscellaneous assets size
page load bundle size
History
To update your PR or re-run it, just comment with: |
…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>
…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>
Summary
triggers_actions_uitotriggersActionsUi#59191