Skip to content

[8.10][Ops] Re-enable typecheck#167609

Merged
delanni merged 4 commits intoelastic:8.10from
delanni:backport/8.10/pr-167392
Oct 2, 2023
Merged

[8.10][Ops] Re-enable typecheck#167609
delanni merged 4 commits intoelastic:8.10from
delanni:backport/8.10/pr-167392

Conversation

@delanni
Copy link
Contributor

@delanni delanni commented Sep 29, 2023

Summary

Semi-manual backport of #167392

This would re-enable full typecheck in the 8.10 branch.

Closes: #167373

…c#167392)

## Summary
This is hopefully the last batch of typescript issues to be fixed,
related to elastic#166813.

It's also re-enabling full typecheck, with this, we should be back in a
clean, typechecked main branch.

Blocked by elastic#167428

---------

Co-authored-by: Brad White <Ikuni17@users.noreply.github.com>
Co-authored-by: Brad White <brad.white@elastic.co>
Co-authored-by: Thomas Watson <watson@elastic.co>
Co-authored-by: Patryk Kopyciński <contact@patrykkopycinski.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 8a29a5e)

# Conflicts:
#	.buildkite/pipelines/pull_request/base.yml
#	.buildkite/scripts/steps/check_types_commits.sh
@delanni delanni added backport This PR is a backport of another PR chore Team:Operations Kibana-Operations Team release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v8.10.3 labels Sep 29, 2023
@elasticmachine
Copy link
Contributor

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

@delanni delanni requested a review from watson September 29, 2023 09:26
Copy link
Contributor

@watson watson left a comment

Choose a reason for hiding this comment

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

🎉

@delanni delanni changed the title [Backport][Ops] Re-enable typecheck [8.10][Ops] Re-enable typecheck Sep 29, 2023
delanni added a commit that referenced this pull request Sep 29, 2023
## Summary
Fix a few issues that occurred around
`x-pack/plugins/security_solution_ess/tsconfig.json` in 8.10.

- chore(typecheck): fix type errors re: unused variables and missing
mock fields

Required for: #167609 

Part of: #167373
@delanni
Copy link
Contributor Author

delanni commented Oct 2, 2023

@elasticmachine merge upstream

@delanni delanni enabled auto-merge (squash) October 2, 2023 14:25
@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@delanni delanni merged commit bdf0dc0 into elastic:8.10 Oct 2, 2023
@delanni delanni deleted the backport/8.10/pr-167392 branch November 11, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting backport This PR is a backport of another PR chore release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v8.10.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants