Skip to content

Conversation

@fc7
Copy link
Contributor

@fc7 fc7 commented Aug 6, 2015

This pull request contains, besides more or less minor code cleanup and refactoring, a bugfix for the problem with customized entry types (commit e92238a). See https://sourceforge.net/p/jabref/bugs/1221/ (with many duplicates)

@fc7 fc7 changed the title Dev 2.11 Fix customized entry problem Aug 6, 2015
@koppor
Copy link
Member

koppor commented Aug 6, 2015

Looks good from a first sight. Two minor things: missing CHANGELOD entry and tabs instead of spaces (maybe IDE formatter_settings.xml are not enabled). I would opt for merging this PR, adding a CHANGELOG entry and let the formatter task do its work.

@fc7
Copy link
Contributor Author

fc7 commented Aug 7, 2015

Thanks, I will take care of my formatter settings for future commits. Concerning the CHANGELOG, do you generally also include bugfixes in it? If not, this PR does not contain any new feature.

@simonharrer
Copy link
Contributor

Yes, please include bugfixes as well.

koppor added a commit that referenced this pull request Aug 7, 2015
Fix customized entry problem
@koppor koppor merged commit eccd973 into JabRef:dev_2.11 Aug 7, 2015
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.

3 participants