Skip to content

src: change env.h includes for forward declarations when possible

Rodrigo Muino Tomonari requested to merge github/fork/alferpal/cleanup/reduce_number_of_includes_env_and_env-inl into master

Due to how the Environment class is used through the codebase, there are a lot of includes referencing either env.h or env-inl.h.

This can cause that when any development touches those libraries, a lot of files have to be recompiled.

This commit attempts to change those includes by forward declarations when possible to mitigate the issue.

Partial fix for #27531 (closed)

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

First PR in node, so if anything can be done better, please, tell me.

Merge request reports