Skip to content

Conversation

Siedlerchr
Copy link
Member

@Siedlerchr Siedlerchr commented Sep 11, 2016

So, I tested all kind of changes locally and it worked.
Even inside Eclipse it showed the correct names

Followup from #1945

@grimes2 Could you please test the changes, too?
In a couple of minutes hopefully the new version is online when travis passes
http://builds.jabref.org/gradleEncoding/

@Siedlerchr
Copy link
Member Author

I tried the installer and for me it looks good:
jabrefgradlewin7utf8

@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Sep 11, 2016
@grimes2
Copy link
Contributor

grimes2 commented Sep 11, 2016

Good news! It works!

Could you please change in DEVELOPERS file Joerg Lenhard to Jörg (to make it perfect).
Thanks for your good work.

JabRef 3.7dev--snapshot--2016-09-11--gradleEncoding--1581dd9
windows 10 10.0 amd64
Java 1.8.0_101

@tobiasdiez
Copy link
Member

Code LGTM.

@Siedlerchr Siedlerchr merged commit ee0f842 into master Sep 11, 2016
@Siedlerchr Siedlerchr deleted the gradleEncoding branch September 11, 2016 20:02
@Siedlerchr Siedlerchr restored the gradleEncoding branch September 11, 2016 20:03
@Siedlerchr Siedlerchr deleted the gradleEncoding branch September 11, 2016 20:12
@koppor koppor mentioned this pull request Sep 12, 2016
5 tasks
zesaro pushed a commit to zesaro/jabref that referenced this pull request Nov 22, 2016
* Add filteringCharset = 'UTF-8'
Convert tabs to spaces
* Read properties file with UTF8 encoding
* Joerg->Jörg
* Unified spaces
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.

3 participants