New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extract core extensions into a Utils module #2112
Conversation
@@ -83,7 +83,9 @@ def self.configuration(override) | |||
config = config.read_config_files(config.config_files(override)) | |||
|
|||
# Merge DEFAULTS < _config.yml < override | |||
config = config.deep_merge(override).stringify_keys | |||
config = Utils.hash_stringify_keys( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps we could use stringify_hash_keys instead? It seems to read better at least. stringify_keys would also be good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Totally fine to rename the functions – stringify_hash_keys
is a bit cleaner.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only struck through it because you already released the new version of the sass converter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can release as many versions as we want! If it's better for future maintainability, let's switch it up. If it were manual labour and incredibly difficult, I'd say let's ignore it, but I have yet to break a sweat coding so it can't be that hard. :)
❤️ |
Fixes #2111