Skip to content

🏗 Enable type-checking for performance and get-zindex#34661

Merged
rsimha merged 2 commits intoampproject:mainfrom
rsimha:2021-06-02-PerformanceTypeCheck
Jun 3, 2021
Merged

🏗 Enable type-checking for performance and get-zindex#34661
rsimha merged 2 commits intoampproject:mainfrom
rsimha:2021-06-02-PerformanceTypeCheck

Conversation

@rsimha
Copy link
Copy Markdown
Contributor

@rsimha rsimha commented Jun 2, 2021

PR Highlights:

  • Enable type-checking for build-system/tasks/performance/
  • Enable type-checking for build-system/tasks/get-zindex
  • Convert lazy requires to regular requires (okay because subpackage deps are now auto-installed)
  • Add a few typedefs, fix a few type errors

With this, there are zero pending directories for which type-checking needs to be enabled.

Future work:

Fixes #28387

@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Jun 2, 2021

Hey @rileyajones! These files were changed:

build-system/tsconfig.json

@rsimha rsimha changed the title 🏗 Enable type-checking for build-system/tasks/performance 🏗 Enable type-checking for performance and get-zindex Jun 3, 2021
@rsimha rsimha requested a review from alanorozco June 3, 2021 00:21
@rsimha
Copy link
Copy Markdown
Contributor Author

rsimha commented Jun 3, 2021

Adding reviewers based on recent edits:

Copy link
Copy Markdown
Contributor

@rileyajones rileyajones left a comment

Choose a reason for hiding this comment

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

This looks great, thanks!

@rsimha rsimha enabled auto-merge (squash) June 3, 2021 00:42
@rsimha rsimha merged commit 52d720b into ampproject:main Jun 3, 2021
@rsimha rsimha deleted the 2021-06-02-PerformanceTypeCheck branch June 3, 2021 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add JSDoc enforcement + some sort of type checking for build-system/

2 participants