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

Additional information on bibtex file #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mistune
Copy link

@mistune mistune commented Jul 1, 2022

  1. Include abstract and DOI when exporting bibtex file.
  2. Using utf-8 encoding fixes encoding errors on a Windows 10, Python 3 setup.

2. Using utf-8 encoding fixes encoding errors on a Windows 10, Python 3 setup.
@jonatasgrosman
Copy link
Owner

jonatasgrosman commented Sep 19, 2022

Hi @ZeroCommits, thanks for your contribution.
I've seen that you remove the __version__ variable, but this is necessary for the cli:version command. Even if you could remove this variable, the right way would be to open another PR for it

@mistune
Copy link
Author

mistune commented Sep 19, 2022

Hi @jonatasgrosman sorry about that! I only intended to share commit 76ea698. Let me do some readings on how to revert the other changes.

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.

2 participants