Skip to content

Improve parser warnings#4702

Merged
Siedlerchr merged 1 commit into
masterfrom
improvWarnings
Feb 27, 2019
Merged

Improve parser warnings#4702
Siedlerchr merged 1 commit into
masterfrom
improvWarnings

Conversation

@tobiasdiez

Copy link
Copy Markdown
Member

Based on http://discourse.jabref.org/t/warnings-by-jabref-start/1532, in which a user (rightfully) complained that the error message is not very helpful.


  • Change in CHANGELOG.md described
  • Tests created for changes
  • Manually tested changed features in running JabRef
  • Screenshots added in PR description (for bigger UI changes)
  • Ensured that the git commit message is a good one
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

Based on http://discourse.jabref.org/t/warnings-by-jabref-start/1532, in which a user (rightfully) complained that the error message is not very helpful.
@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Feb 26, 2019
@Siedlerchr Siedlerchr merged commit 3cf609c into master Feb 27, 2019
@Siedlerchr Siedlerchr deleted the improvWarnings branch February 27, 2019 07:27
Siedlerchr added a commit that referenced this pull request Mar 1, 2019
…tofx

* upstream/master:
  Bump fontbox from 2.0.13 to 2.0.14 (#4705)
  Bump pdfbox from 2.0.13 to 2.0.14 (#4706)
  Bump xmpbox from 2.0.13 to 2.0.14 (#4704)
  Improve parser warnings (#4702)
  Bump checkstyle from 8.17 to 8.18 (#4700)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants