Skip to content

[compiler] Validate environment config while parsing plugin opts#30726

Merged
josephsavona merged 3 commits into
gh/josephsavona/37/basefrom
gh/josephsavona/37/head
Aug 17, 2024
Merged

[compiler] Validate environment config while parsing plugin opts#30726
josephsavona merged 3 commits into
gh/josephsavona/37/basefrom
gh/josephsavona/37/head

Conversation

@josephsavona

@josephsavona josephsavona commented Aug 16, 2024

Copy link
Copy Markdown
Contributor

Stack from ghstack (oldest at bottom):

Addresses a todo from a while back. We now validate environment options when parsing the plugin options, which means we can stop re-parsing/validating in later phases.

Addresses a todo from a while back. We now validate environment options when parsing the plugin options, which means we can stop re-parsing/validating in later phases.

[ghstack-poisoned]
@vercel

vercel Bot commented Aug 16, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 16, 2024 11:50pm

josephsavona added a commit that referenced this pull request Aug 16, 2024
Addresses a todo from a while back. We now validate environment options when parsing the plugin options, which means we can stop re-parsing/validating in later phases.

ghstack-source-id: c835586
Pull Request resolved: #30726
@facebook-github-bot facebook-github-bot added the React Core Team Opened by a member of the React Core Team label Aug 16, 2024
…n opts"

Addresses a todo from a while back. We now validate environment options when parsing the plugin options, which means we can stop re-parsing/validating in later phases.

[ghstack-poisoned]
josephsavona added a commit that referenced this pull request Aug 16, 2024
Addresses a todo from a while back. We now validate environment options when parsing the plugin options, which means we can stop re-parsing/validating in later phases.

ghstack-source-id: acb52a9
Pull Request resolved: #30726
@josephsavona josephsavona requested a review from poteto August 16, 2024 21:04
…n opts"

Addresses a todo from a while back. We now validate environment options when parsing the plugin options, which means we can stop re-parsing/validating in later phases.

[ghstack-poisoned]
@josephsavona josephsavona merged commit 2758f27 into gh/josephsavona/37/base Aug 17, 2024
josephsavona added a commit that referenced this pull request Aug 17, 2024
Addresses a todo from a while back. We now validate environment options when parsing the plugin options, which means we can stop re-parsing/validating in later phases.

ghstack-source-id: b19806e
Pull Request resolved: #30726
@josephsavona josephsavona deleted the gh/josephsavona/37/head branch August 17, 2024 00:05
eps1lon added a commit to vercel/next.js that referenced this pull request Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants