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

Integration assistant generate readme #192887

Merged
merged 10 commits into from
Sep 17, 2024

Conversation

haetamoudi
Copy link
Contributor

@haetamoudi haetamoudi commented Sep 13, 2024

Summary

  • Create an integration README when using the Integration Assistant so that it is displayed when opening the newly created integration.
  • Increase code coverage for the Integration Assistant plugin.

Context

Related issue: #192704

Result

  • When using the Integration Assistant flow, the integration README is now automatically created.
  • Currently, the README is identical to the one found in _dev/build/docs. A follow-up PR will enhance the content of the README based on the requirements outlined in issue #192704.
Screenshot 2024-09-12 at 15 45 00

Code coverage of the integration assistant plugin is improved.

Screenshot 2024-09-13 at 15 59 56

Checklist

Delete any items that are not applicable to this PR.

@haetamoudi haetamoudi added enhancement New value added to drive a business result release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) v8.16.0 Team:Security-Scalability Team label for Security Integrations Scalability Team v8.15.2 labels Sep 13, 2024
@haetamoudi haetamoudi linked an issue Sep 13, 2024 that may be closed by this pull request
4 tasks
@haetamoudi haetamoudi marked this pull request as ready for review September 13, 2024 15:55
@haetamoudi haetamoudi requested a review from a team as a code owner September 13, 2024 15:55
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-scalability (Team:Security-Scalability)

Copy link
Contributor

@kgeller kgeller left a comment

Choose a reason for hiding this comment

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

LGTM

@haetamoudi haetamoudi added backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development and removed backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Sep 16, 2024
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

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

cc @haetamoudi

@haetamoudi haetamoudi merged commit e360963 into elastic:main Sep 17, 2024
25 checks passed
@haetamoudi haetamoudi deleted the automatic-import-generate-readme branch September 17, 2024 16:31
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 17, 2024
## Summary

- Create an integration `README` when using the Integration Assistant so
that it is displayed when opening the newly created integration.
- Increase code coverage for the Integration Assistant plugin.

(cherry picked from commit e360963)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 17, 2024
## Summary

- Create an integration `README` when using the Integration Assistant so
that it is displayed when opening the newly created integration.
- Increase code coverage for the Integration Assistant plugin.

(cherry picked from commit e360963)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.15
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Sep 17, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [Integration assistant generate readme
(#192887)](#192887)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Hanna
Tamoudi","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-17T16:31:18Z","message":"Integration
assistant generate readme (#192887)\n\n## Summary\r\n\r\n- Create an
integration `README` when using the Integration Assistant so\r\nthat it
is displayed when opening the newly created integration.\r\n- Increase
code coverage for the Integration Assistant
plugin.","sha":"e360963065ab783c9cf265869fb43b2f73f74368","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["enhancement","release_note:skip","v9.0.0","backport:prev-major","v8.16.0","Team:Security-Scalability","v8.15.2","Feature:AutomaticImport"],"title":"Integration
assistant generate
readme","number":192887,"url":"https://github.com/elastic/kibana/pull/192887","mergeCommit":{"message":"Integration
assistant generate readme (#192887)\n\n## Summary\r\n\r\n- Create an
integration `README` when using the Integration Assistant so\r\nthat it
is displayed when opening the newly created integration.\r\n- Increase
code coverage for the Integration Assistant
plugin.","sha":"e360963065ab783c9cf265869fb43b2f73f74368"}},"sourceBranch":"main","suggestedTargetBranches":["8.x","8.15"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/192887","number":192887,"mergeCommit":{"message":"Integration
assistant generate readme (#192887)\n\n## Summary\r\n\r\n- Create an
integration `README` when using the Integration Assistant so\r\nthat it
is displayed when opening the newly created integration.\r\n- Increase
code coverage for the Integration Assistant
plugin.","sha":"e360963065ab783c9cf265869fb43b2f73f74368"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.15","label":"v8.15.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Hanna Tamoudi <[email protected]>
kibanamachine added a commit that referenced this pull request Sep 17, 2024
# Backport

This will backport the following commits from `main` to `8.15`:
- [Integration assistant generate readme
(#192887)](#192887)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Hanna
Tamoudi","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-17T16:31:18Z","message":"Integration
assistant generate readme (#192887)\n\n## Summary\r\n\r\n- Create an
integration `README` when using the Integration Assistant so\r\nthat it
is displayed when opening the newly created integration.\r\n- Increase
code coverage for the Integration Assistant
plugin.","sha":"e360963065ab783c9cf265869fb43b2f73f74368","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["enhancement","release_note:skip","v9.0.0","backport:prev-major","v8.16.0","Team:Security-Scalability","v8.15.2","Feature:AutomaticImport"],"title":"Integration
assistant generate
readme","number":192887,"url":"https://github.com/elastic/kibana/pull/192887","mergeCommit":{"message":"Integration
assistant generate readme (#192887)\n\n## Summary\r\n\r\n- Create an
integration `README` when using the Integration Assistant so\r\nthat it
is displayed when opening the newly created integration.\r\n- Increase
code coverage for the Integration Assistant
plugin.","sha":"e360963065ab783c9cf265869fb43b2f73f74368"}},"sourceBranch":"main","suggestedTargetBranches":["8.x","8.15"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/192887","number":192887,"mergeCommit":{"message":"Integration
assistant generate readme (#192887)\n\n## Summary\r\n\r\n- Create an
integration `README` when using the Integration Assistant so\r\nthat it
is displayed when opening the newly created integration.\r\n- Increase
code coverage for the Integration Assistant
plugin.","sha":"e360963065ab783c9cf265869fb43b2f73f74368"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.15","label":"v8.15.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Hanna Tamoudi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development enhancement New value added to drive a business result Feature:AutomaticImport release_note:skip Skip the PR/issue when compiling release notes Team:Security-Scalability Team label for Security Integrations Scalability Team v8.15.2 v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Automatic Import] Add package ReadMe
6 participants