fix(inject): rollup is optional peer dependency

[milahu]

Rollup Plugin Name: inject

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• no

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

Relevant issues:

• Make rollup an optional peer dependency climateinteractive/SDEverywhere#209
• missing peer dependencies after installation of pnpm v6.24.4 pnpm/pnpm#4183

Description

this plugin can be used from vite
for example in https://github.com/sodatea/vite-plugin-node-stdlib-browser

in this case, rollup is not installed

this fix should also be applied to other rollup plugins

workaround for pnpm: add this to package.json

{
  "pnpm": {
    "peerDependencyRules": {
      "ignoreMissing": [
        "rollup"
      ]
    }
  }
}

[lukastaegert]

Thanks a lot! Considering your arguments, I added this change to all PRs I just created for Rollup 3 compatibility of plugins, see e.g. #1292. This supersedes your PR, but we can still merge it first if you want the contribution.

[milahu]

thanks : )

This supersedes your PR, but we can still merge it first if you want the contribution.

i dont care