Skip to content

Commit

Permalink
Remove "modules/" autoload
Browse files Browse the repository at this point in the history
  • Loading branch information
@timkelty
timkelty committed Aug 27, 2024
1 parent 8b56598 commit 1c73e64
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
5 changes: 0 additions & 5 deletions composer.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,6 @@
"craftcms/generator": "^1.3.0",
"yiisoft/yii2-shell": "^2.0.3"
},
"autoload": {
"psr-4": {
"modules\\": "modules/"
}
},
"config": {
"allow-plugins": {
"craftcms/plugin-installer": true,
Expand Down
5 changes: 0 additions & 5 deletions composer.json.default
View file
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,6 @@
"craftcms/generator": "^1.3.0",
"yiisoft/yii2-shell": "^2.0.3"
},
"autoload": {
"psr-4": {
"modules\\": "modules/"
}
},
"config": {
"allow-plugins": {
"craftcms/plugin-installer": true,
Expand Down

2 comments on commit 1c73e64

@mmikkel
Copy link

@mmikkel mmikkel commented on 1c73e64 Sep 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@timkelty This broke our bootstrapper/boilerplate; any particular reason why the autoload map was removed?

@timkelty
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mmikkel See craftcms/generator#37

It hasn't been bootstrapped by app.php for quite some time, so it didn't make sense to keep it in the starter, and was causing problems when people tried to use craft make module.

Please sign in to comment.