magento / devdocs

[Deprecated] Magento Developer Documentation
Open Software License 3.0
674 stars 1.76k forks source link

app/etc/config.php tracking in VCS - docs conflict with links to .gitignore #9194

Closed df2k2 closed 3 months ago

df2k2 commented 3 years ago

Track app/etc/config.php or not?

Topic Link

https://devdocs.magento.com/guides/v2.4/config-guide/prod/config-reference-gitignore.html

Description

The references to .gitignore found here: https://devdocs.magento.com/guides/v2.4/config-guide/prod/config-reference-gitignore.html and https://devdocs.magento.com/guides/v2.3/config-guide/prod/config-reference-gitignore.html are:

Which contain a reference to ignore app/etc/config.php : https://github.com/magento/magento2/blob/2.4-develop/.gitignore#L18

However, there are several statements throughout devdocs that would lead to an assumption that this app/etc/config.php file would not be set for ignore by VCS in .gitignore for 2.3 and 2.4 CE:

Reference 1: https://devdocs.magento.com/guides/v2.4/config-guide/deployment/pipeline/production-system.html

  • Make sure all of the following are included in source control:
    • app/etc/config.php

Reference 2: https://devdocs.magento.com/guides/v2.4/config-guide/deployment/pipeline/example/shared-configuration.html

Generate the shared configuration file, app/etc/config.php, in your development system and transfer it using source control to your build system as discussed in this section

Reference 3: https://devdocs.magento.com/guides/v2.4/config-guide/deployment/pipeline/technical-details.html

Manage the shared configuration The shared configuration is stored in app/etc/config.php, which should be in source control.

Reference 4: https://devdocs.magento.com/guides/v2.4/config-guide/prod/config-reference-most.html

The magento app:config:dump command writes these values to the shared configuration file, app/etc/config.php, which should be in source control.

Possible solutions

One of:

Additional information

m2-assistant[bot] commented 3 years ago

Hi @df2k2. Thank you for your report. To help us process this issue please make sure that you provided sufficient information.

Please, add a comment to assign the issue: @magento I am working on this


hostep commented 3 years ago

It was reported on the magento2 repo to remove the config.php file again from the .gitignore file, but no idea if something will be done with it, the issue has been open for quite a while now and is a bit messy (the messy part is probably my fault 😛)

erikmarr commented 3 years ago

@jfrontain can you take a look?

dshevtsov commented 3 months ago

The topic was migrated. If the issue is still relevant, please feel free to reopen it in https://github.com/AdobeDocs/commerce-operations.en/issues.