Skip to content

Conversation

@shairez
Copy link
Contributor

@shairez shairez commented Nov 18, 2025

PR Checklist

Originally in #1892 I've put the resolving of the dynamic tsconfig in configResolved thinking it'll run after the nx vite plugin configResolved because their plugin is configured as "pre".

But no, the analog vite plugin runs before and because of that the first compilation is ignoring the dynamic value and because of that the loading times are slower.

What is the new behavior?

Moved it to the performCompilation function, which definitally runs after nx has set the env variable which is needed to resovle the dynamic tsconfig.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

[optional] What gif best describes this PR or how it makes you feel?

@netlify
Copy link

netlify bot commented Nov 18, 2025

Deploy Preview for analog-app ready!

Name Link
🔨 Latest commit 3cb4988
🔍 Latest deploy log https://app.netlify.com/projects/analog-app/deploys/691ca4acac4c1e000829ed61
😎 Deploy Preview https://deploy-preview-1959--analog-app.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Nov 18, 2025

Deploy Preview for analog-docs ready!

Name Link
🔨 Latest commit 3cb4988
🔍 Latest deploy log https://app.netlify.com/projects/analog-docs/deploys/691ca4acc321770008b7ec72
😎 Deploy Preview https://deploy-preview-1959--analog-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Nov 18, 2025

Deploy Preview for analog-blog ready!

Name Link
🔨 Latest commit 3cb4988
🔍 Latest deploy log https://app.netlify.com/projects/analog-blog/deploys/691ca4ac64ced2000801c18e
😎 Deploy Preview https://deploy-preview-1959--analog-blog.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@brandonroberts brandonroberts merged commit 04dd8f3 into analogjs:beta Nov 18, 2025
18 checks passed
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.

2 participants