Skip to content

feat: Improve plugin dependency requirements#270

Open
simondotm wants to merge 3 commits intomainfrom
sm-cleaner-deps
Open

feat: Improve plugin dependency requirements#270
simondotm wants to merge 3 commits intomainfrom
sm-cleaner-deps

Conversation

@simondotm
Copy link
Copy Markdown
Owner

@simondotm simondotm commented Jan 23, 2026

We're running unnecessary init generator calls in function & sync generators that ensure packages exist in the workspace.
This has a side effect of adding dependencies that users may not want in their workspace.

This PR:

  • Removes calls to init generator for functions + sync (on the assumption that app generator has to have previously run, and deps were added at that point)
  • Removes plugin addition of firebase-functions-test dependency. The plugin doesn't require it, and it's optional for firebase development, so this way users are free to add it themselves if/when they need to

Fixes #269

@simondotm simondotm changed the title feat: Don't call init generator in function/sync generator feat: Improve plugin dependency requirements Jan 23, 2026
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.

Installs unnecessary dependencies

1 participant