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
Implement metadata mapping #2820
Conversation
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
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.
LGTM
nikola/utils.py
Outdated
"""Map metadata from other platforms to Nikola names. | ||
|
||
This uses the METADATA_MAPPING setting (via ``config``) and modifies the dict in place.""" | ||
for foreign, ours in config['METADATA_MAPPING'].get(key, {}).items(): |
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.
config
can be None
, when _get_metadata_from_file
is called without config
specified.
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.
Fair point (although it’s very unlikely). Fixed on master.
METADATA_MAPPING
setting (Issue Map Pelican/Hugo metadata to Nikola names #2817)