Skip to content
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

Add publication_list plugin to easily manage publication list. #128

Merged
merged 1 commit into from Jan 25, 2016

Conversation

xuhdev
Copy link
Member

@xuhdev xuhdev commented Jan 24, 2016

Add a plugin to manage publication list.

@xuhdev
Copy link
Member Author

xuhdev commented Jan 24, 2016

The reason the test failed on Travis CI is not caused by this PR. It is caused by something wrong with automated installation on dependency packages.

@xuhdev xuhdev force-pushed the publication_list branch 2 times, most recently from faa3693 to e81a3e5 Compare January 24, 2016 09:43
This plugin provides a directive `publication-list`, which reads a bib file and
display the references in them on the web page. The `publication-list` directive
accepts one option `:style:`, which is the style of the bibliography. All
available styles are provided by Pybtex. You can see the [list of styles]() in
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Broken link.

You can see the [list of styles][] in the Pybtex repository.

@Kwpolska
Copy link
Member

The documentation (plugin description, README.md) needs a heavy sprinkling of the name BibTeX.

To make the test pass, merge/rebase on master, and fix two PEP 8 complaints:

v7/publication_list/publication_list.py:64:65: E251 unexpected spaces around keyword / parameter equals
v7/publication_list/publication_list.py:64:67: E251 unexpected spaces around keyword / parameter equals

Signed-off-by: Hong Xu <hong@topbug.net>
@xuhdev
Copy link
Member Author

xuhdev commented Jan 24, 2016

@Kwpolska Done.

Kwpolska added a commit that referenced this pull request Jan 25, 2016
Add publication_list plugin to easily manage publication list.
@Kwpolska Kwpolska merged commit 65209c7 into getnikola:master Jan 25, 2016
@Kwpolska
Copy link
Member

Thanks for your contribution!

@xuhdev xuhdev deleted the publication_list branch January 26, 2016 10:55
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.

None yet

2 participants