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 re2 library #186155

Closed
wants to merge 3 commits into from
Closed

Conversation

pgayvallet
Copy link
Contributor

Summary

After #186104, get rid of the last usage of re2 in code and in tooling

@pgayvallet
Copy link
Contributor Author

/ci

@pgayvallet pgayvallet added release_note:skip Skip the PR/issue when compiling release notes v8.15.0 labels Jun 13, 2024
RE2_DOWNLOAD_MIRROR:
'https://us-central1-elastic-kibana-184716.cloudfunctions.net/kibana-ci-proxy-cache/node-re2',
},
env: {},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

kept the entry, tell me if I should remove env: {} altogether.

},
},
];
const packages: Package[] = [];
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Kept the empty array and the task, in case we may need to re-add more native modules at some point.

Please tell me if I should fully remove that task instead.

@pgayvallet
Copy link
Contributor Author

/ci

@pgayvallet pgayvallet marked this pull request as ready for review June 13, 2024 13:27
@pgayvallet pgayvallet requested review from a team as code owners June 13, 2024 13:27
@mistic
Copy link
Member

mistic commented Jun 13, 2024

@pgayvallet I will pickup this PR next week and evolve it in the lines of https://github.com/elastic/kibana-operations/issues/131 as I've agreed this yesterday with Luke and Brandon

@pgayvallet
Copy link
Contributor Author

@mistic sounds good. Will you take over my PR or should I close it?

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #62 / Dev Tools Search Profiler Editor supports pre-configured search query

Metrics [docs]

✅ unchanged

History

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

Copy link
Contributor

@nickofthyme nickofthyme left a comment

Choose a reason for hiding this comment

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

Vis changes LGTM

@mistic
Copy link
Member

mistic commented Jun 14, 2024

@pgayvallet I will cherrypick what you have on your branch but will probably open a new one as I don't know when I will fully complete it (probably somewhere next week)

@pgayvallet
Copy link
Contributor Author

Sounds good - closing then

@pgayvallet pgayvallet closed this Jun 14, 2024
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 v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants