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

refactor(core): Polish ahead of time ts implementation #34773

Merged
merged 10 commits into from
Feb 11, 2022

Conversation

tyhopp
Copy link
Contributor

@tyhopp tyhopp commented Feb 9, 2022

Description

  • Make .cache/compile an exported constant we can reuse
  • Use reporter activity timer for gatsby file compilation
  • Add tests for gatsby file compilation

Related Issues

[sc-45862]

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Feb 9, 2022
@tyhopp tyhopp marked this pull request as draft February 9, 2022 06:32
@tyhopp tyhopp added topic: core Relates to Gatsby's core (e.g. page loading, reporter, state machine) and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Feb 9, 2022
@tyhopp tyhopp changed the title Refactor ts aot refactor(core): Polish ahead of time ts implementation Feb 9, 2022
@imjoshin
Copy link
Contributor

imjoshin commented Feb 9, 2022

These tests are 🔥

@LekoArts LekoArts marked this pull request as ready for review February 11, 2022 12:36
@LekoArts LekoArts merged commit 9c78852 into feat/ts-aot Feb 11, 2022
@LekoArts LekoArts deleted the refactor-ts-aot branch February 11, 2022 12:57
gatsbybot pushed a commit that referenced this pull request Feb 28, 2022
* get simplest gatsby-config.ts working

* make gatsby-node work

* use program.directory for default-site-plugin for browser/ssr

* chore(docs): update TS doc (#34757)

Co-authored-by: LekoArts <lekoarts@gmail.com>

* feat(gatsby-parcel-config): create parcel config package (#34779)

Co-authored-by: Lennart <lekoarts@gmail.com>

* refactor(core): Polish ahead of time ts implementation (#34773)

Co-authored-by: LekoArts <lekoarts@gmail.com>

* tmp

* update parcel deps version

* update snapshot

* merge master

* feat(core): TS compilation of local plugin gatsby files (#34827)

Co-authored-by: LekoArts <lekoarts@gmail.com>

* Update scaling-issues.md

* Restore build.js

* Restore develop.js

* Restore repl.js

* set flag to always true

* chore(docs): update main tutorial to incorporate TS Updates to create-gatsby (#34890)

Co-authored-by: Lennart <lekoarts@gmail.com>

* chore(gatsby): compile only TS files (#34892)

* use require resolve for defaultConfig

* update doc to add limitations and migration guides

* add structured error logging

* fix cache reslience setup (#34915)

* bump @parcel/source-map in monorepo lock

* adjust tests so cache dirs are cleared only before compilations to workaround EBUSY problems on Windows

* address review comments

* docs: specify gatsby versions supporting particular TS features (#34945)

Co-authored-by: Lennart <lekoarts@gmail.com>

Co-authored-by: Jude Agboola <marvinjudehk@gmail.com>
Co-authored-by: Ty Hopp <tyhopp@users.noreply.github.com>
Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
Co-authored-by: gatsbybot <mathews.kyle+gatsbybot@gmail.com>
@LekoArts LekoArts added this to the TypeScript Improvements milestone Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: core Relates to Gatsby's core (e.g. page loading, reporter, state machine)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants