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

Fix tests in maintable beta except L10n #3863

Merged
merged 1 commit into from
Mar 19, 2018
Merged

Conversation

Siedlerchr
Copy link
Member

@Siedlerchr Siedlerchr commented Mar 18, 2018

I fixed some of the failing tests, except the l10n tests.
I am unsure what we do with the L10N issues. Let's discuss this in a devcall.


  • 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?)

@Siedlerchr Siedlerchr changed the base branch from master to maintable-beta March 18, 2018 16:10
@Siedlerchr Siedlerchr added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers status: devcall labels Mar 18, 2018
@Siedlerchr Siedlerchr merged commit 06d6736 into maintable-beta Mar 19, 2018
@Siedlerchr Siedlerchr deleted the fixTests branch March 19, 2018 19:30
Siedlerchr added a commit that referenced this pull request Mar 19, 2018
…esDlg

* upstream/maintable-beta:
  fix failing tests in maintable beta (#3863)
@Siedlerchr Siedlerchr added this to the v5.0 milestone Apr 20, 2018
@Siedlerchr
Copy link
Member Author

Dev Calll decision: Postponed to when 5.0 will be master

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.

4 participants