Since we're calling config.fetch invariably without handling nil, its better that we set the optional parameter as an empty hash lest a NoMethodError: undefined method 'fetch' for nil:NilClass be raised.
The reason why this remained concealed all this while is because Jekyll::Site automatically initializes all Jekyll::Generator subclasses with site.config. Therefore, this change is not going to cause any impact whatsoever, but is correcting a wrong nevertheless.
mattr-
commented
5 years ago
Since we're calling
config.fetchinvariably without handlingnil, its better that we set the optional parameter as an empty hash lest aNoMethodError: undefined method 'fetch' for nil:NilClassbe raised.The reason why this remained concealed all this while is because
Jekyll::Siteautomatically initializes allJekyll::Generatorsubclasses withsite.config. Therefore, this change is not going to cause any impact whatsoever, but is correcting a wrong nevertheless.