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

Use default value for process.env.WP_SRC_DIRECTORY. #44365

Closed
wants to merge 1 commit into from

Conversation

loxK
Copy link
Contributor

@loxK loxK commented Sep 22, 2022

What?

Fixes #44309

Why?

See discussion here: #43917 (review)

How?

Using wp-scripts command, if is not specified, process.env.WP_SRC_DIRECTORY defaults to string 'src'. When extending the webpack.config.js directly, it should behave the same.

@github-actions
Copy link

👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @loxK! In case you missed it, we'd love to have you join us in our Slack community, where we hold regularly weekly meetings open to anyone to coordinate with each other.

If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information.

@github-actions github-actions bot added the First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository label Sep 22, 2022
@gziolo gziolo added the [Tool] WP Scripts /packages/scripts label Sep 22, 2022
@loxK loxK closed this Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository [Tool] WP Scripts /packages/scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scripts: regression - webpack config cannot be extended anymore
2 participants