Skip to content

♻️ Enable passing type-checking on src/context #34387

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

Merged
merged 24 commits into from
May 17, 2021

Conversation

rcebulko
Copy link
Contributor

Part of #34096

@rcebulko rcebulko requested review from samouri and caroqliu May 14, 2021 17:04
@amp-owners-bot amp-owners-bot bot requested a review from danielrozenberg May 14, 2021 17:04
Copy link
Member

@danielrozenberg danielrozenberg left a comment

Choose a reason for hiding this comment

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

Approval for build-system/*

Copy link
Member

@samouri samouri left a comment

Choose a reason for hiding this comment

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

Really impressive. This is not easy code to work with

*/
let UsedDef;
function UsedDef() {}
Copy link
Member

Choose a reason for hiding this comment

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

FMI: did you have to make this format change away from typedef due to incompatibility with templates?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes

@rcebulko
Copy link
Contributor Author

Merging since @caroqliu is OOO all week, but pinging for retroactive visibility

@rcebulko rcebulko merged commit 8cb5da6 into ampproject:main May 17, 2021
@rcebulko rcebulko deleted the types-context branch May 17, 2021 17:33
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.

4 participants