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

Remove eslint-plugin-prefer-object-spread dependency #162439

Merged

Conversation

watson
Copy link
Contributor

@watson watson commented Jul 24, 2023

Remove the eslint-plugin-prefer-object-spread dependency as the prefer-object-spread rule has been part of ESLint since v5 (we currently run ESLint v7).

@watson watson added Team:Operations Team label for Operations Team release_note:skip Skip the PR/issue when compiling release notes v7.17.10 v7.17.12 labels Jul 24, 2023
@watson watson self-assigned this Jul 24, 2023
@watson watson requested review from a team as code owners July 24, 2023 19:39
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

cc @watson

Copy link
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

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

Visualizations team changes LGTM

@watson watson merged commit a79e9c7 into elastic:main Jul 25, 2023
@watson watson deleted the remove-eslint-plugin-prefer-object-spread branch July 25, 2023 08:04
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 25, 2023
Remove the `eslint-plugin-prefer-object-spread` dependency as the
`prefer-object-spread` rule has been part of ESLint since v5

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

💔 Some backports could not be created

Status Branch Result
7.17 Backport failed because of merge conflicts
8.9

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 162439

Questions ?

Please refer to the Backport tool documentation

@watson watson removed the v7.17.10 label Jul 25, 2023
watson pushed a commit to watson/kibana that referenced this pull request Jul 25, 2023
Remove the `eslint-plugin-prefer-object-spread` dependency as the
`prefer-object-spread` rule has been part of ESLint since v5

(cherry picked from commit a79e9c7)

# Conflicts:
#	packages/kbn-tinymath/src/index.js
@watson
Copy link
Contributor Author

watson commented Jul 25, 2023

💚 All backports created successfully

Status Branch Result
7.17

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 Jul 25, 2023
…162455)

# Backport

This will backport the following commits from `main` to `8.9`:
- [Remove eslint-plugin-prefer-object-spread dependency
(#162439)](#162439)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Thomas
Watson","email":"watson@elastic.co"},"sourceCommit":{"committedDate":"2023-07-25T08:04:44Z","message":"Remove
eslint-plugin-prefer-object-spread dependency (#162439)\n\nRemove the
`eslint-plugin-prefer-object-spread` dependency as
the\r\n`prefer-object-spread` rule has been part of ESLint since
v5","sha":"a79e9c737498bc3f3bda3d82ef1b3387be88b9ed","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","release_note:skip","v7.17.10","v8.10.0","v7.17.12"],"number":162439,"url":"https://github.com/elastic/kibana/pull/162439","mergeCommit":{"message":"Remove
eslint-plugin-prefer-object-spread dependency (#162439)\n\nRemove the
`eslint-plugin-prefer-object-spread` dependency as
the\r\n`prefer-object-spread` rule has been part of ESLint since
v5","sha":"a79e9c737498bc3f3bda3d82ef1b3387be88b9ed"}},"sourceBranch":"main","suggestedTargetBranches":["7.17"],"targetPullRequestStates":[{"branch":"7.17","label":"v7.17.10","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/162439","number":162439,"mergeCommit":{"message":"Remove
eslint-plugin-prefer-object-spread dependency (#162439)\n\nRemove the
`eslint-plugin-prefer-object-spread` dependency as
the\r\n`prefer-object-spread` rule has been part of ESLint since
v5","sha":"a79e9c737498bc3f3bda3d82ef1b3387be88b9ed"}}]}] BACKPORT-->

Co-authored-by: Thomas Watson <watson@elastic.co>
watson pushed a commit that referenced this pull request Jul 25, 2023
…#162461)

# Backport

This will backport the following commits from `main` to `7.17`:
- [Remove eslint-plugin-prefer-object-spread dependency
(#162439)](#162439)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Thomas
Watson","email":"watson@elastic.co"},"sourceCommit":{"committedDate":"2023-07-25T08:04:44Z","message":"Remove
eslint-plugin-prefer-object-spread dependency (#162439)\n\nRemove the
`eslint-plugin-prefer-object-spread` dependency as
the\r\n`prefer-object-spread` rule has been part of ESLint since
v5","sha":"a79e9c737498bc3f3bda3d82ef1b3387be88b9ed","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","release_note:skip","v8.10.0","v7.17.12"],"number":162439,"url":"https://github.com/elastic/kibana/pull/162439","mergeCommit":{"message":"Remove
eslint-plugin-prefer-object-spread dependency (#162439)\n\nRemove the
`eslint-plugin-prefer-object-spread` dependency as
the\r\n`prefer-object-spread` rule has been part of ESLint since
v5","sha":"a79e9c737498bc3f3bda3d82ef1b3387be88b9ed"}},"sourceBranch":"main","suggestedTargetBranches":["7.17"],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/162439","number":162439,"mergeCommit":{"message":"Remove
eslint-plugin-prefer-object-spread dependency (#162439)\n\nRemove the
`eslint-plugin-prefer-object-spread` dependency as
the\r\n`prefer-object-spread` rule has been part of ESLint since
v5","sha":"a79e9c737498bc3f3bda3d82ef1b3387be88b9ed"}},{"branch":"7.17","label":"v7.17.12","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"url":"https://github.com/elastic/kibana/pull/162455","number":162455,"branch":"8.9","state":"OPEN"}]}]
BACKPORT-->
@mistic mistic added v8.9.1 and removed v8.9.0 labels Jul 25, 2023
@mistic
Copy link
Member

mistic commented Jul 25, 2023

This PR didn't make it into the latest BC of 8.9.0 and 7.17.12. Updating the labels.

@mistic mistic added v7.17.13 and removed v7.17.12 labels Jul 25, 2023
ThomThomson pushed a commit to ThomThomson/kibana that referenced this pull request Aug 1, 2023
Remove the `eslint-plugin-prefer-object-spread` dependency as the
`prefer-object-spread` rule has been part of ESLint since v5
@watson watson mentioned this pull request Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.17.13 v8.9.1 v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants