Skip to content

🐛 fix: allow autoloading of modules with no config#23

Merged
Cambis merged 1 commit into1.xfrom
fix/allow-no-config-modules
Jun 30, 2025
Merged

🐛 fix: allow autoloading of modules with no config#23
Cambis merged 1 commit into1.xfrom
fix/allow-no-config-modules

Conversation

@Cambis
Copy link
Copy Markdown
Owner

@Cambis Cambis commented Jun 30, 2025

Description ✍️

  • Currently we are checking if vendor modules have the _config.php or _config file in order to autoload their files.
  • Modules don't need this anymore, they just need to be a silverstripe-vendormodule or silverstripe-theme. I've decided to not include the themes as you shouldn't be including php in those anyways

DoD checklist ✅

- This only applies if you run phpstan from within inside a module
@Cambis Cambis marked this pull request as ready for review June 30, 2025 09:31
@Cambis Cambis merged commit 1fcd450 into 1.x Jun 30, 2025
19 checks passed
@Cambis Cambis deleted the fix/allow-no-config-modules branch June 30, 2025 09:31
Cambis added a commit that referenced this pull request Jun 30, 2025
- This only applies if you run phpstan from within inside a module

Signed-off-by: Cameron Bryers <12287346+Cambis@users.noreply.github.com>
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.

1 participant