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 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 xuhdev:publication_list branch 2 times, most recently from faa3693 to e81a3e5 Jan 24, 2016
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

This comment has been minimized.

@Kwpolska

Kwpolska Jan 24, 2016
Member

Broken link.

You can see the [list of styles][] in the Pybtex repository.
@Kwpolska
Copy link
Member

@Kwpolska Kwpolska commented Jan 24, 2016

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 xuhdev force-pushed the xuhdev:publication_list branch from e81a3e5 to 0fe9aa8 Jan 24, 2016
@xuhdev
Copy link
Member Author

@xuhdev 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
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Kwpolska
Copy link
Member

@Kwpolska Kwpolska commented Jan 25, 2016

Thanks for your contribution!

@xuhdev xuhdev deleted the xuhdev:publication_list branch Jan 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants