Skip to content

Fix issues when a config.rb file is present somewhere in the directories#281

Merged
plusjade merged 2 commits intoruhoh:masterfrom
sztupy:fix-config-rb
Mar 29, 2014
Merged

Fix issues when a config.rb file is present somewhere in the directories#281
plusjade merged 2 commits intoruhoh:masterfrom
sztupy:fix-config-rb

Conversation

@sztupy
Copy link
Copy Markdown
Contributor

@sztupy sztupy commented Mar 2, 2014

In this case the config.yml or config.json files might not get parsed, resulting in missing configuration values, and/or errors

…ories

In this case the `config.yml` or `config.json` files might not get parsed, resulting in missing configuration
@sztupy
Copy link
Copy Markdown
Contributor Author

sztupy commented Mar 3, 2014

It seems it also fixes the issue with the config.yml file not being picked up in the default generated blog (where there is a config.ru file in the root)

@indolent-gnathostome
Copy link
Copy Markdown

Much appreciated, @sztupy; especially the new test. It looks like changes to lib/ruhoh/config.rb in this commit are relevant to #257, #256, #254, and #251.

@sztupy
Copy link
Copy Markdown
Contributor Author

sztupy commented Mar 4, 2014

Ah, found a bug with loading the config files from the subdirectories. Fixed and added tests for it

plusjade added a commit that referenced this pull request Mar 29, 2014
Fix issues when a `config.rb` file is present somewhere in the directories
@plusjade plusjade merged commit 1c7df99 into ruhoh:master Mar 29, 2014
@plusjade
Copy link
Copy Markdown
Member

@sztupy much appreciated! Sorry I haven't been around lately and it has taken me a while to merge this in.

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.

3 participants