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

Update Cytoscape (main) - abandoned #205444

Closed
wants to merge 2 commits into from

Conversation

elastic-renovate-prod[bot]
Copy link
Contributor

This PR contains the following updates:

Package Type Update Change
@types/cytoscape (source) devDependencies minor ^3.14.0 -> ^3.21.8
cytoscape (source) dependencies minor ^3.10.0 -> ^3.30.4
cytoscape-dagre dependencies minor ^2.2.2 -> ^2.5.0

Release Notes

cytoscape/cytoscape.js (cytoscape)

v3.30.4

Compare Source

Release version v3.30.4

v3.30.3

Compare Source

v3.30.2

Compare Source

Release version v3.30.2

v3.30.1

Compare Source

Release version v3.30.1

v3.30.0

Compare Source

Release version v3.30.0

v3.29.3

Compare Source

Release version v3.29.3

v3.29.2

Compare Source

Release version v3.29.2

v3.29.1

Compare Source

Release version v3.29.1

v3.29.0

Compare Source

Release version v3.29.0

v3.28.1

Compare Source

Release version v3.28.1

v3.28.0

Compare Source

Release version v3.28.0

v3.27.2

Compare Source

Release version v3.27.2

v3.27.1

Compare Source

Release version v3.27.1

v3.27.0

Compare Source

Release version v3.27.0

v3.26.2

Compare Source

Release version v3.26.2

v3.26.1

Compare Source

Release version v3.26.1

v3.26.0

Compare Source

v3.25.2

Compare Source

Release version v3.25.2

v3.25.1

Compare Source

v3.25.0

Compare Source

v3.24.2

Compare Source

v3.24.1

Compare Source

v3.24.0

Compare Source

v3.23.1

Compare Source

v3.23.0

Compare Source

v3.22.1

Compare Source

v3.22.0

Compare Source

v3.21.3

Compare Source

v3.21.2

Compare Source

v3.21.1

Compare Source

v3.21.0

Compare Source

v3.20.3

Compare Source

v3.20.2

Compare Source

v3.20.1

Compare Source

v3.20.0

Compare Source

v3.19.1

Compare Source

v3.19.0

Compare Source

v3.18.3

Compare Source

v3.18.2

Compare Source

v3.18.1

Compare Source

v3.18.0

Compare Source

v3.17.4

Compare Source

v3.17.3

Compare Source

v3.17.2

Compare Source

v3.17.1

Compare Source

v3.17.0

Compare Source

v3.16.5

Compare Source

v3.16.4

Compare Source

v3.16.3

Compare Source

v3.16.2

Compare Source

v3.16.1

Compare Source

v3.16.0

Compare Source

v3.15.5

Compare Source

v3.15.4

Compare Source

v3.15.3

Compare Source

v3.15.2

Compare Source

v3.15.1

Compare Source

v3.15.0

Compare Source

v3.14.4

Compare Source

v3.14.3

Compare Source

v3.14.2

Compare Source

v3.14.1

Compare Source

v3.14.0

Compare Source

v3.13.3

Compare Source

v3.13.2

Compare Source

v3.13.1

Compare Source

v3.13.0

Compare Source

v3.12.3

Compare Source

v3.12.2

Compare Source

v3.12.1

Compare Source

v3.12.0

Compare Source

v3.11.2

Compare Source

v3.11.1

Compare Source

v3.11.0

Compare Source

v3.10.2

Compare Source

v3.10.1

Compare Source

cytoscape/cytoscape.js-dagre (cytoscape-dagre)

v2.5.0

Compare Source

v2.4.0

Compare Source

v2.3.2

Compare Source

v2.3.1

Compare Source

v2.3.0

Compare Source


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@elastic-renovate-prod elastic-renovate-prod bot added backport:all-open Backport to all branches that could still receive a release release_note:skip Skip the PR/issue when compiling release notes labels Jan 2, 2025
@elastic-renovate-prod elastic-renovate-prod bot requested a review from a team January 2, 2025 20:55
Copy link
Member

@jennypavlova jennypavlova left a comment

Choose a reason for hiding this comment

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

LGTM (Code review only)

@elastic-renovate-prod
Copy link
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@Bluefinger
Copy link
Contributor

/ci

@elasticmachine
Copy link
Contributor

⏳ Build in-progress, with failures

Failed CI Steps

@Bluefinger
Copy link
Contributor

It seems the update contained a semver breaking change in the typing for the packages (I love semver breakages). We can't merge this PR without some changes.

Bluefinger added a commit that referenced this pull request Jan 8, 2025
…205726)

## Summary

Updates Cytoscape to newer versions, requiring one change with some
`removeListener` usage no longer being valid typing.

Supersedes #205444

## How to test

- Passes CI with no type errors or failed CI jobs for ML
- Job Map or wherever cytoscape is being used on ML doesn't leak event
listeners.
- Usages in APM also do not break.
@Bluefinger
Copy link
Contributor

Closed in favour of #205726

@Bluefinger Bluefinger closed this Jan 8, 2025
@elastic-renovate-prod elastic-renovate-prod bot changed the title Update Cytoscape (main) Update Cytoscape (main) - abandoned Jan 8, 2025
@elastic-renovate-prod
Copy link
Contributor Author

Autoclosing Skipped

This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

Bluefinger added a commit to Bluefinger/kibana that referenced this pull request Jan 8, 2025
…lastic#205726)

## Summary

Updates Cytoscape to newer versions, requiring one change with some
`removeListener` usage no longer being valid typing.

Supersedes elastic#205444

## How to test

- Passes CI with no type errors or failed CI jobs for ML
- Job Map or wherever cytoscape is being used on ML doesn't leak event
listeners.
- Usages in APM also do not break.

(cherry picked from commit 7363f03)

# Conflicts:
#	x-pack/solutions/observability/plugins/apm/public/components/app/service_map/cytoscape_options.ts
#	yarn.lock
Bluefinger added a commit to Bluefinger/kibana that referenced this pull request Jan 8, 2025
…lastic#205726)

## Summary

Updates Cytoscape to newer versions, requiring one change with some
`removeListener` usage no longer being valid typing.

Supersedes elastic#205444

## How to test

- Passes CI with no type errors or failed CI jobs for ML
- Job Map or wherever cytoscape is being used on ML doesn't leak event
listeners.
- Usages in APM also do not break.

(cherry picked from commit 7363f03)

# Conflicts:
#	package.json
#	x-pack/plugins/observability_solution/apm/public/components/app/service_map/cytoscape_options.ts
#	yarn.lock
Bluefinger added a commit to Bluefinger/kibana that referenced this pull request Jan 8, 2025
…lastic#205726)

## Summary

Updates Cytoscape to newer versions, requiring one change with some
`removeListener` usage no longer being valid typing.

Supersedes elastic#205444

## How to test

- Passes CI with no type errors or failed CI jobs for ML
- Job Map or wherever cytoscape is being used on ML doesn't leak event
listeners.
- Usages in APM also do not break.

(cherry picked from commit 7363f03)

# Conflicts:
#	package.json
#	x-pack/plugins/observability_solution/apm/public/components/app/service_map/cytoscape_options.ts
#	yarn.lock
Bluefinger added a commit to Bluefinger/kibana that referenced this pull request Jan 8, 2025
…lastic#205726)

## Summary

Updates Cytoscape to newer versions, requiring one change with some
`removeListener` usage no longer being valid typing.

Supersedes elastic#205444

## How to test

- Passes CI with no type errors or failed CI jobs for ML
- Job Map or wherever cytoscape is being used on ML doesn't leak event
listeners.
- Usages in APM also do not break.
Bluefinger added a commit that referenced this pull request Jan 8, 2025
…ons (#205726) (#205848)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ML][APM] Fix Cytoscape listener typing, Update Cytoscape versions
(#205726)](#205726)

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

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

<!--BACKPORT [{"author":{"name":"Gonçalo Rica Pais da
Silva","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-08T09:16:17Z","message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:all-open","Team:obs-ux-infra_services"],"title":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape
versions","number":205726,"url":"https://github.com/elastic/kibana/pull/205726","mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205726","number":205726,"mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}}]}]
BACKPORT-->
@Bluefinger Bluefinger closed this Jan 8, 2025
Bluefinger added a commit that referenced this pull request Jan 8, 2025
…ions (#205726) (#205850)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[ML][APM] Fix Cytoscape listener typing, Update Cytoscape versions
(#205726)](#205726)

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

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

<!--BACKPORT [{"author":{"name":"Gonçalo Rica Pais da
Silva","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-08T09:16:17Z","message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:all-open","Team:obs-ux-infra_services"],"title":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape
versions","number":205726,"url":"https://github.com/elastic/kibana/pull/205726","mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205726","number":205726,"mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}}]}]
BACKPORT-->
Bluefinger added a commit that referenced this pull request Jan 8, 2025
…ions (#205726) (#205854)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[ML][APM] Fix Cytoscape listener typing, Update Cytoscape versions
(#205726)](#205726)

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

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

<!--BACKPORT [{"author":{"name":"Gonçalo Rica Pais da
Silva","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-08T09:16:17Z","message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:all-open","Team:obs-ux-infra_services"],"title":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape
versions","number":205726,"url":"https://github.com/elastic/kibana/pull/205726","mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205726","number":205726,"mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}}]}]
BACKPORT-->
Bluefinger added a commit that referenced this pull request Jan 8, 2025
…ions (#205726) (#205867)

# Backport

This will backport the following commits from `main` to `7.17`:
- [[ML][APM] Fix Cytoscape listener typing, Update Cytoscape versions
(#205726)](#205726)

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

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

<!--BACKPORT [{"author":{"name":"Gonçalo Rica Pais da
Silva","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-08T09:16:17Z","message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:all-open","Team:obs-ux-infra_services"],"title":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape
versions","number":205726,"url":"https://github.com/elastic/kibana/pull/205726","mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205726","number":205726,"mergeCommit":{"message":"[ML][APM]
Fix Cytoscape listener typing, Update Cytoscape versions (#205726)\n\n##
Summary\r\n\r\nUpdates Cytoscape to newer versions, requiring one change
with some\r\n`removeListener` usage no longer being valid
typing.\r\n\r\nSupersedes #205444\r\n\r\n## How to test\r\n\r\n- Passes
CI with no type errors or failed CI jobs for ML\r\n- Job Map or wherever
cytoscape is being used on ML doesn't leak event\r\nlisteners.\r\n-
Usages in APM also do not
break.","sha":"7363f03d81835f70f39caa27e7096b8a0361ae69"}}]}]
BACKPORT-->
crespocarlos pushed a commit to crespocarlos/kibana that referenced this pull request Jan 8, 2025
…lastic#205726)

## Summary

Updates Cytoscape to newer versions, requiring one change with some
`removeListener` usage no longer being valid typing.

Supersedes elastic#205444

## How to test

- Passes CI with no type errors or failed CI jobs for ML
- Job Map or wherever cytoscape is being used on ML doesn't leak event
listeners.
- Usages in APM also do not break.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:all-open Backport to all branches that could still receive a release release_note:skip Skip the PR/issue when compiling release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants