Momo Kornher

Great lead @matt9ucci

@blimmer Is this option really so much better than using a low-level patch? I'm worried we are locking ourselves into an API that we then have to spend extra work...

`package.json` does something similar. The steps are basically: - Load the current plugin data from the file - Synth the file, including the current plugin data - Establish if plugin...

Last PRs are up!

All done now. 🎉

Yikes, this is bad. I think the only reason we have this is for the tests in projen itself. For which we absolutely do not have to use `NODE_ENV`.

@hs-kgoriunov Apologies, but the change has to be reverted. This unintentionally broke the documented approach to testing custom project types (https://projen.io/docs/testing/#testing) Bug report: https://github.com/projen/projen/discussions/3322#discussion-6178633 `NODE_ENV` is the only way to...

> To my understanding, there are two Projen flavors—Projen as a tool and Projen as a framework. The first one shouldn't get impacted by the environment, while the second should....

But I think your point is the following: - `npx projen ci` defines a task `yarn ci` - projen doesn't do anything but passing through to the actual command -...

Thanks for the detailed explanation. Your pseudo code seems to imply that we should have a production build version that has this stuff disabled (or ripped out). To be clear,...