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

Removes kbn-std/unset as it's not used anywhere #193298

Merged
merged 2 commits into from
Sep 20, 2024
Merged

Conversation

rudolf
Copy link
Contributor

@rudolf rudolf commented Sep 18, 2024

Summary

Removes unset as it's not used anywhere. Underscore includes an unset function which most plugins seem to prefer.

@rudolf rudolf added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc release_note:skip Skip the PR/issue when compiling release notes labels Sep 18, 2024
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/std 72 70 -2

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-srcJs 3.4MB 3.4MB -229.0B
Unknown metric groups

API count

id before after diff
@kbn/std 111 108 -3

History

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

@rudolf rudolf marked this pull request as ready for review September 19, 2024 12:51
@rudolf rudolf requested a review from a team as a code owner September 19, 2024 12:51
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

Copy link
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

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

Please modify the PR description and remove parts that you don't need!

@rudolf rudolf added the backport:all-open Backport to all branches that could still receive a release label Sep 20, 2024
@rudolf rudolf merged commit bddeac0 into elastic:main Sep 20, 2024
27 of 28 checks passed
@rudolf rudolf deleted the rm-unset branch September 20, 2024 13:44
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 20, 2024
## Summary

Removes unset as it's not used anywhere. Underscore includes an unset
function which most plugins seem to prefer.

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

💔 Some backports could not be created

Status Branch Result
7.17 Backport failed because of merge conflicts
8.15 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.15:
- [Security Solution] [Attack discovery] Includes the user.target.name field in the default Anonymization allow list to improve Attack discoveries (#193496)
- fix(slo): use correct uisettings (#193362)
8.x

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 193298

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Sep 20, 2024
…193585)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Removes kbn-std/unset as it's not used anywhere
(#193298)](#193298)

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

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

<!--BACKPORT [{"author":{"name":"Rudolf
Meijering","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-20T13:44:52Z","message":"Removes
kbn-std/unset as it's not used anywhere (#193298)\n\n##
Summary\r\n\r\nRemoves unset as it's not used anywhere. Underscore
includes an unset\r\nfunction which most plugins seem to
prefer.","sha":"bddeac062a7fc1c87cacca0ef2c94ea2bd22f5c3","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","release_note:skip","v9.0.0","backport:all-open"],"title":"Removes
kbn-std/unset as it's not used
anywhere","number":193298,"url":"https://github.com/elastic/kibana/pull/193298","mergeCommit":{"message":"Removes
kbn-std/unset as it's not used anywhere (#193298)\n\n##
Summary\r\n\r\nRemoves unset as it's not used anywhere. Underscore
includes an unset\r\nfunction which most plugins seem to
prefer.","sha":"bddeac062a7fc1c87cacca0ef2c94ea2bd22f5c3"}},"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/193298","number":193298,"mergeCommit":{"message":"Removes
kbn-std/unset as it's not used anywhere (#193298)\n\n##
Summary\r\n\r\nRemoves unset as it's not used anywhere. Underscore
includes an unset\r\nfunction which most plugins seem to
prefer.","sha":"bddeac062a7fc1c87cacca0ef2c94ea2bd22f5c3"}}]}]
BACKPORT-->

Co-authored-by: Rudolf Meijering <[email protected]>
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 Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants