-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Ownership] Assign test files to visualizations team #200144
[Ownership] Assign test files to visualizations team #200144
Conversation
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.
Some comments from me
.github/CODEOWNERS
Outdated
@@ -1111,7 +1138,11 @@ src/plugins/discover/public/context_awareness/profile_providers/security @elasti | |||
/test/api_integration/apis/event_annotations @elastic/kibana-visualizations | |||
/x-pack/test_serverless/functional/test_suites/common/visualizations/ @elastic/kibana-visualizations | |||
/x-pack/test_serverless/functional/fixtures/kbn_archiver/lens/ @elastic/kibana-visualizations | |||
packages/kbn-monaco/src/esql @elastic/kibana-esql | |||
packages/kbn-monaco/src/esql |
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.
Why did you remove our ownership from here? 🤔
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.
ahh, mistake! apologies Stratoula!
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.
.github/CODEOWNERS
Outdated
/x-pack/test/functional/apps/managed_content @elastic/kibana-visualizations | ||
/test/functional/services/visualizations @elastic/kibana-visualizations | ||
/test/functional/services/renderable.ts @elastic/kibana-visualizations | ||
/test/functional/services/inspector.ts @elastic/kibana-visualizations # Assigned per https://github.com/elastic/kibana/pull/28509 |
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.
The PR you mention is of 2019 😄 Inspector is owned by the presentation team nowadays
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.
whoops! things have changed since I joined! hahahaa
Thanks Stratoula!
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.
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.
ES|QL tests ownership LGTM
LEt's wait for the vis team to check theirs before merging
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.
Reviewed and looks good to me!
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
History
|
## Summary Assign test files to visualizations team Contributes to: elastic#192979 --------- Co-authored-by: Stratoula Kalafateli <[email protected]>
## Summary Assign test files to visualizations team Contributes to: elastic#192979 --------- Co-authored-by: Stratoula Kalafateli <[email protected]>
Summary
Assign test files to visualizations team
Contributes to: #192979