Skip to content

[ts] Check .d.ts files for all projects in typeCheck#114295

Merged
spalger merged 6 commits intoelastic:masterfrom
spalger:implement/check-lib-files-on-ci
Oct 13, 2021
Merged

[ts] Check .d.ts files for all projects in typeCheck#114295
spalger merged 6 commits intoelastic:masterfrom
spalger:implement/check-lib-files-on-ci

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Oct 7, 2021

In #107981 I stopped composite projects from being executed with --skip-lib-check=false by only testing types in the build ts refs step when using node scripts/type_check. This PR reverts that change and (along with #114342) fixes the newly broken types since this change was merged.

[skip ci]

@spalger spalger force-pushed the implement/check-lib-files-on-ci branch from fdd1eb0 to 784b4fa Compare October 7, 2021 18:29
@spalger spalger force-pushed the implement/check-lib-files-on-ci branch 4 times, most recently from c86bc9d to 3837cab Compare October 11, 2021 17:46
@spalger spalger changed the title enable --skip-lib-check for all projects in typeCheck and fix issues [ts] Check .d.ts files for all projects in typeCheck Oct 11, 2021
@spalger spalger added release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.16.0 v8.0.0 labels Oct 11, 2021
@spalger spalger force-pushed the implement/check-lib-files-on-ci branch from 3837cab to 3a4a992 Compare October 11, 2021 20:23
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@spalger spalger marked this pull request as ready for review October 12, 2021 16:21
@spalger spalger requested review from a team as code owners October 12, 2021 16:21
@elasticmachine
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

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

Code LGTM

Copy link
Copy Markdown
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

Reporting change LGTM

@spalger spalger merged commit 6c5dd08 into elastic:master Oct 13, 2021
@spalger spalger deleted the implement/check-lib-files-on-ci branch October 13, 2021 15:45
@spalger spalger added the auto-backport Deprecated - use backport:version if exact versions are needed label Oct 13, 2021
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Oct 13, 2021
* enable --skip-lib-check for all projects in typeCheck and fix existing issues

* fix graph types

* transpile TS to ES2019, but not all the way back to es5

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Oct 13, 2021
* enable --skip-lib-check for all projects in typeCheck and fix existing issues

* fix graph types

* transpile TS to ES2019, but not all the way back to es5

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Spencer <email@spalger.com>
Co-authored-by: spalger <spalger@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed buildkite-ci release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.16.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants