posts module now reads config values from config module #26

Merged
cmccabe merged 1 commits from refactor_posts into master 2019-11-27 11:46:50 +00:00
Collaborator

This is further work on issues #14 and #17.

Configuration values, like the location of the data files, is now stored in module config. This means that the values can be configured once, and then reused in every module.

This is similar to the way that Bombadillo structures configuration, which has more features like user configuration, and system default configuration.

Currently only the data directory and data file are configured, and are used only in the posts module. The glob path is also present, as a possible idea for issue #25.

Let me know what you think!

This is further work on issues #14 and #17. Configuration values, like the location of the data files, is now stored in module config. This means that the values can be configured once, and then reused in every module. This is similar to the way that Bombadillo structures configuration, which has more features like user configuration, and system default configuration. Currently only the data directory and data file are configured, and are used only in the posts module. The glob path is also present, as a possible idea for issue #25. Let me know what you think!
asdf reviewed 2019-11-27 02:05:58 +00:00
Author
Collaborator

So, instead of defining the data file name each time you perform some operation, you can import config.CONFIG and use the value from there. If it ever needs to change, it can be changed in once place.

So, instead of defining the data file name each time you perform some operation, you can import config.CONFIG and use the value from there. If it ever needs to change, it can be changed in once place.
Collaborator

This is a good change. I dig it. 👍

This is a good change. I dig it. :thumbsup:
cmccabe closed this pull request 2019-11-27 11:46:49 +00:00
asdf deleted branch refactor_posts 2019-11-29 07:56:56 +00:00
Sign in to join this conversation.
No description provided.