-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Misc TypeScript Migrations #6500
Conversation
Gatsby Cloud Build Reportethereum-org-website-dev 🎉 Your build was successful! See the Deploy preview here. Build Details🕐 Build time: 16m PerformanceLighthouse report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @WilliamWelsh, thanks for the PR 😎
I'm a TypeScript noob so I'll leave it to @pettinarip to double-check this. I've left a few open questions that I'm unsure of (though I might be entirely wrong 🙂)
I can't comment directly on the file but for the eth-upgrade-articles
I think we should be explicitly declaring types e.g.:
interface Article {
title: string,
description: string,
link: string,
}
You're right, I went ahead and added the article type. I also noticed that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking amazing @WilliamWelsh thanks for tackling so many files. Left a couple of comments there. LMK if you have any question.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@WilliamWelsh a few small comments, the rest is looking great.
Oops Co-authored-by: Pablo Pettinari <[email protected]>
Oops Co-authored-by: Pablo Pettinari <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! thanks for the contribution @WilliamWelsh
@all-contributors please add @WilliamWelsh for code |
I've put up a pull request to add @WilliamWelsh! 🎉 |
I wanted to help with the TypeScript rollout, so I've migrated the following so far:
src/data
src/api
src/lambda
src/utils
Related Issue
#6392