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

[deps] Replace faker with @faker-js #201105

Merged
merged 7 commits into from
Dec 3, 2024

Conversation

markov00
Copy link
Member

@markov00 markov00 commented Nov 21, 2024

Summary

The faker library is not maintained anymore and is replaced by a community fork @faker-js.
This PR migrates all the usages of faker to the new library, trying to use the same methods (even if they have slight differences in results like faker.random.number() has a max of 99999 where instead faker.number.int() have a MAX_SAFE_INTEGER as max).

@markov00 markov00 added release_note:skip Skip the PR/issue when compiling release notes dependencies Pull requests that update a dependency file v9.0.0 backport:version Backport to applied version labels v8.17.0 v8.18.0 labels Nov 21, 2024
@markov00 markov00 marked this pull request as ready for review December 2, 2024 11:08
@markov00 markov00 requested review from a team as code owners December 2, 2024 11:08
@markov00 markov00 requested a review from vitaliidm December 2, 2024 11:08
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Dec 2, 2024
Copy link
Contributor

@tonyghiani tonyghiani left a comment

Choose a reason for hiding this comment

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

logs-shared changes LGTM, code review only 👌

dmlemeshko added a commit that referenced this pull request Dec 2, 2024
…02329)

## Summary

@markov00 opened PR to
[migrate](#201105) from `faker`,
but it probably will take extra time and to keep track let's move its
ownership to `team:kibana-visualizations`

---------

Co-authored-by: Marco Vettorello <[email protected]>
Copy link
Contributor

@PhilippeOberti PhilippeOberti left a comment

Choose a reason for hiding this comment

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

LGTM for the Threat Hunting Investigations team

@markov00 markov00 merged commit b61ad41 into elastic:main Dec 3, 2024
11 checks passed
@markov00 markov00 deleted the 2024_11_21-replace_faker_lib branch December 3, 2024 17:18
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.17, 8.x

https://github.com/elastic/kibana/actions/runs/12144804651

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.17 Backport failed because of merge conflicts
8.x Backport failed because of merge conflicts

You might need to backport the following PRs to 8.x:
- Create the "Asset Inventory" Kibana Plugin (#202291)
- Update dependency @elastic/request-converter to ^8.16.2 (main) (#201856)
- Upgrades cypress to 13.15.1 (#198715)

Manual backport

To create the backport manually run:

node scripts/backport --pr 201105

Questions ?

Please refer to the Backport tool documentation

markov00 added a commit that referenced this pull request Dec 4, 2024
## Summary

With the [replacement](#201105) of
`faker` with `@faker-js` there is no more need of this renovate
configuration.
@markov00 markov00 added backport:skip This commit does not require backporting and removed backport:version Backport to applied version labels v8.17.0 v8.18.0 labels Dec 4, 2024
hop-dev pushed a commit to hop-dev/kibana that referenced this pull request Dec 5, 2024
## Summary

The `faker` library is[ not maintained
anymore](https://fakerjs.dev/about/announcements/2022-01-14.html#i-heard-something-happened-what-s-the-tldr)
and is replaced by a community fork `@faker-js`.
This PR migrates all the usages of faker to the new library, trying to
use the same methods (even if they have slight differences in results
like `faker.random.number()` has a max of 99999 where instead
`faker.number.int()` have a MAX_SAFE_INTEGER as max).
hop-dev pushed a commit to hop-dev/kibana that referenced this pull request Dec 5, 2024
## Summary

With the [replacement](elastic#201105) of
`faker` with `@faker-js` there is no more need of this renovate
configuration.
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Dec 9, 2024
## Summary

With the [replacement](elastic#201105) of
`faker` with `@faker-js` there is no more need of this renovate
configuration.
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Dec 9, 2024
## Summary

With the [replacement](elastic#201105) of
`faker` with `@faker-js` there is no more need of this renovate
configuration.
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 9, 2024
…astic#202329)

## Summary

@markov00 opened PR to
[migrate](elastic#201105) from `faker`,
but it probably will take extra time and to keep track let's move its
ownership to `team:kibana-visualizations`

---------

Co-authored-by: Marco Vettorello <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 9, 2024
## Summary

The `faker` library is[ not maintained
anymore](https://fakerjs.dev/about/announcements/2022-01-14.html#i-heard-something-happened-what-s-the-tldr)
and is replaced by a community fork `@faker-js`.
This PR migrates all the usages of faker to the new library, trying to
use the same methods (even if they have slight differences in results
like `faker.random.number()` has a max of 99999 where instead
`faker.number.int()` have a MAX_SAFE_INTEGER as max).
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 9, 2024
## Summary

With the [replacement](elastic#201105) of
`faker` with `@faker-js` there is no more need of this renovate
configuration.
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
…astic#202329)

## Summary

@markov00 opened PR to
[migrate](elastic#201105) from `faker`,
but it probably will take extra time and to keep track let's move its
ownership to `team:kibana-visualizations`

---------

Co-authored-by: Marco Vettorello <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
## Summary

The `faker` library is[ not maintained
anymore](https://fakerjs.dev/about/announcements/2022-01-14.html#i-heard-something-happened-what-s-the-tldr)
and is replaced by a community fork `@faker-js`.
This PR migrates all the usages of faker to the new library, trying to
use the same methods (even if they have slight differences in results
like `faker.random.number()` has a max of 99999 where instead
`faker.number.int()` have a MAX_SAFE_INTEGER as max).
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
## Summary

With the [replacement](elastic#201105) of
`faker` with `@faker-js` there is no more need of this renovate
configuration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting ci:project-deploy-observability Create an Observability project dependencies Pull requests that update a dependency file release_note:skip Skip the PR/issue when compiling release notes v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.