Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Enable local Twig debugging #86

Merged
merged 1 commit into from
May 27, 2016
Merged

Conversation

dpagini
Copy link
Contributor

@dpagini dpagini commented May 26, 2016

Addresses #85

The below lines show the value of $dir being the project root, and not the docroot folder. Maybe this is a problem with the DRUPAL_ROOT value?
$dir = dirname(DRUPAL_ROOT);
$config_directories['sync'] = $dir . '/config/default';

This PR points to the development.services.yml file in the correct location.

@dpagini
Copy link
Contributor Author

dpagini commented May 26, 2016

Please be gentle, I think this is my first public repo PR. Any pointers are appreciated.

@grasmash
Copy link
Contributor

@dpagini Thanks for catching this. The PR looks good.

$dir should reference the directory above the docroot, so that's working as expected. Perhaps we should rename $dir to $repo_root, which is the terminology used elsewhere in the project.

@grasmash grasmash merged commit 77f5569 into acquia:8.x May 27, 2016
@dpagini
Copy link
Contributor Author

dpagini commented May 27, 2016

Thank you @grasmash, and thanks to @danepowell for helping identify the issue on our project which is utilizing Bolt, a very cool tool.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants