Problem/Motivation
One of these things is not like the others.
Steps to reproduce
install config_distro
install config_sync
go to /admin/modules
See most config modules in the configuration group.
See config_distro, config_filter and config_distro_ignore in the config group
How to test
Have lando installed on your computer
clone this issue fork https://git.drupalcode.org/issue/config_distro-3418198/-/tree/2.0.x
Check out the default 2.0.x branch
Place this lando file into the root of the module (AKA what you cloned)
lando start
lando intall
lando drush uli -l https://config_distro.lndo.site
Now that you have everything set up, you can navigate to the /admin/modules page and view config distro in the configuration group along with the submodules.
name: config_distro
recipe: drupal10
config:
php: '8.2'
via: apache:2.4
webroot: web
database: mariadb:10.4
xdebug: false
services:
database:
type: compose
services:
image: mariadb:10.4
command: docker-entrypoint.sh mariadbd
restart: always
ports:
- '3306'
environment:
MARIADB_ALLOW_EMPTY_ROOT_PASSWORD: 'true'
MARIADB_DATABASE: drupal10
MYSQL_DATABASE: drupal10
MARIADB_USER: drupal10
MARIADB_PASSWORD: drupal10
appserver:
overrides:
environment:
SIMPLETEST_DB: 'mysql://drupal10:drupal10@database/drupal10'
SIMPLETEST_BASE_URL: 'http://appserver'
volumes:
# Don't share our host working directory as /app. We want /app empty for composer.
- /app
# Instead share our host working directory as a standalone package.
- .:/usr/local/config_distro
build:
# Create a new Drupal project and use the module as a non-packagist repository.
- composer create-project --dev drupal/recommended-project:10.2.x /app
- composer config extra.enable-patching true
- composer config extra.composer-exit-on-patch-failure true
- composer config allow-plugins.cweagans/composer-patches true
- composer require cweagans/composer-patches
- composer config minimum-stability dev
- composer config allow-plugins.phpstan/extension-installer true
- composer require --dev drupal/core-dev:^10.1 drush/drush phpspec/prophecy-phpunit:* phpstan/extension-installer mglaman/phpstan-drupal phpstan/phpstan-deprecation-rules
- composer config repositories.localdev path /usr/local/config_distro && composer require drupal/config_distro:\*@dev
tooling:
# Provide a command to install Drupal.
install:
service: appserver
cmd:
- /app/vendor/bin/drush --root=/app/web site:install --account-mail=noreply@example.com --account-name=admin --account-pass=admin --db-url=mysql://drupal10:drupal10@database:3306/drupal10 -y --verbose
- /app/vendor/bin/drush en -y config_distro
# Provide Drush tooling to automatically know the Drupal root.
drush:
service: appserver
cmd: /app/vendor/bin/drush --root=/app/web
phpcs:
service: appserver
cmd: /app/vendor/bin/phpcs -s --colors --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml web/modules/contrib/config_distro
# Provide PHPCBF tooling to fix coding standards.
phpcbf:
service: appserver
cmd: /app/vendor/bin/phpcbf -s --colors --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml web/modules/contrib/config_distro
# Provide phpstan tooling to check for code quality and deprecated code.
phpstan:
service: appserver
cmd: /app/vendor/bin/phpstan analyse --configuration web/modules/contrib/config_distro/phpstan.neon web/modules/contrib/config_distro
# Provide phpunit tooling to run unit tests.
phpunit:
service: appserver
cmd: /app/vendor/bin/phpunit --configuration /app/web/core/phpunit.xml.dist --bootstrap /app/web/core/tests/bootstrap.php /app/web/modules/contrib/config_distro
Proposed resolution
Change the group these three modules are in to Configuration instead of Config
Remaining tasks
Review
User interface changes
When enabling modules config distro, config_filter, and config_distro_ignore are in the configuration group.
API changes
N/A
Data model changes
N/A