Skip to content

[9.0] [scout] enable eslint plugin playwright (#210666)#211172

Merged
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-210666
Feb 14, 2025
Merged

[9.0] [scout] enable eslint plugin playwright (#210666)#211172
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-210666

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

## Summary

Closes elastic#21030

It was a quick work to do while we don't have many tests yet.

For reviewers: we most likely will review and update the rules to align
better with final test design for Scout before GA. I don't think we have
to deep dive into what rules are missing, but just to make sure I didn't
restrict something important from your perspective.

Rules are described in
https://github.com/playwright-community/eslint-plugin-playwright?tab=readme-ov-file#rules

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit ddd0fdb)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Feb 14, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) February 14, 2025 09:27
@kibanamachine kibanamachine merged commit c6ae829 into elastic:9.0 Feb 14, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/scout 0 1 +1

Total ESLint disabled count

id before after diff
@kbn/scout 0 1 +1

cc @dmlemeshko

dmlemeshko added a commit that referenced this pull request Mar 18, 2025
# Backport

This will backport the following commits from `main` to `8.x`:
- [[scout] enable eslint plugin playwright
(#210666)](#210666)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Dzmitry
Lemechko","email":"dzmitry.lemechko@elastic.co"},"sourceCommit":{"committedDate":"2025-02-14T09:21:49Z","message":"[scout]
enable eslint plugin playwright (#210666)\n\n## Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/21030\r\n\r\nIt was a quick
work to do while we don't have many tests yet. \r\n\r\nFor reviewers: we
most likely will review and update the rules to align\r\nbetter with
final test design for Scout before GA. I don't think we have\r\nto deep
dive into what rules are missing, but just to make sure I
didn't\r\nrestrict something important from your
perspective.\r\n\r\nRules are described
in\r\nhttps://github.com/playwright-community/eslint-plugin-playwright?tab=readme-ov-file#rules\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"ddd0fdb4794b71f7c500f5d7d03a846bf21174b9","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","test:scout","v9.1.0","v8.19.0"],"title":"[scout]
enable eslint plugin
playwright","number":210666,"url":"https://github.com/elastic/kibana/pull/210666","mergeCommit":{"message":"[scout]
enable eslint plugin playwright (#210666)\n\n## Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/21030\r\n\r\nIt was a quick
work to do while we don't have many tests yet. \r\n\r\nFor reviewers: we
most likely will review and update the rules to align\r\nbetter with
final test design for Scout before GA. I don't think we have\r\nto deep
dive into what rules are missing, but just to make sure I
didn't\r\nrestrict something important from your
perspective.\r\n\r\nRules are described
in\r\nhttps://github.com/playwright-community/eslint-plugin-playwright?tab=readme-ov-file#rules\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"ddd0fdb4794b71f7c500f5d7d03a846bf21174b9"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/211172","number":211172,"state":"MERGED","mergeCommit":{"sha":"c6ae8292a1a8d80a29966234e98f341ff489a89d","message":"[9.0]
[scout] enable eslint plugin playwright (#210666) (#211172)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`9.0`:\n- [[scout] enable eslint plugin
playwright\n(#210666)](https://github.com/elastic/kibana/pull/210666)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n\n\nCo-authored-by:
Dzmitry Lemechko
<dzmitry.lemechko@elastic.co>"}},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210666","number":210666,"mergeCommit":{"message":"[scout]
enable eslint plugin playwright (#210666)\n\n## Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/21030\r\n\r\nIt was a quick
work to do while we don't have many tests yet. \r\n\r\nFor reviewers: we
most likely will review and update the rules to align\r\nbetter with
final test design for Scout before GA. I don't think we have\r\nto deep
dive into what rules are missing, but just to make sure I
didn't\r\nrestrict something important from your
perspective.\r\n\r\nRules are described
in\r\nhttps://github.com/playwright-community/eslint-plugin-playwright?tab=readme-ov-file#rules\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"ddd0fdb4794b71f7c500f5d7d03a846bf21174b9"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants