Skip to content
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

chore(gatsby): migrate date to TypeScript #22430

Merged
merged 4 commits into from
Jul 15, 2020

Conversation

sasurau4
Copy link
Contributor

Description

Part of #21995

I migrated types/date.js to typescript.

Documentation

Related Issues

#21995

@LekoArts
Copy link
Contributor

LekoArts commented Jun 3, 2020

Sorry for the late response on this. Can you please rebase onto master? Thanks!

@sasurau4 sasurau4 force-pushed the migrate/type-of-date-to-typescript branch from 62a05a6 to b5eaf03 Compare June 5, 2020 01:16
Copy link
Contributor

@pvdz pvdz left a comment

Choose a reason for hiding this comment

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

Hey. I'm sorry this has been untouched in a while. I think it looks good and just needs a few unrelated changes to be dropped to be merged. Do you want to proceed with that? Cheers!

I can try to run the format and rebase bot and see how that ends up. Formatting fine now, Please just drop that added require line and to move the type definitions up :)

packages/gatsby/src/schema/types/__tests__/date.js Outdated Show resolved Hide resolved
packages/gatsby/src/schema/infer/inference-metadata.ts Outdated Show resolved Hide resolved
packages/gatsby/src/schema/types/date.ts Outdated Show resolved Hide resolved
packages/gatsby/src/schema/types/date.ts Outdated Show resolved Hide resolved
@sasurau4 sasurau4 requested review from a team as code owners July 15, 2020 05:20
@sasurau4 sasurau4 force-pushed the migrate/type-of-date-to-typescript branch from 98fd3c2 to 0221560 Compare July 15, 2020 05:24
@sasurau4 sasurau4 requested review from pvdz and removed request for a team July 15, 2020 05:26
Copy link
Contributor

@pvdz pvdz left a comment

Choose a reason for hiding this comment

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

Looks great, thank you! 💜

@pvdz pvdz merged commit f1b3a66 into gatsbyjs:master Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants