Skip to content

Update to JDK 19#9421

Merged
Siedlerchr merged 7 commits into
mainfrom
update_jdk
Dec 26, 2022
Merged

Update to JDK 19#9421
Siedlerchr merged 7 commits into
mainfrom
update_jdk

Conversation

@calixtus

@calixtus calixtus commented Dec 3, 2022

Copy link
Copy Markdown
Member

Fixes #9417

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

Comment thread build.gradle
java {
sourceCompatibility = JavaVersion.VERSION_18
targetCompatibility = JavaVersion.VERSION_18
sourceCompatibility = JavaVersion.VERSION_19

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would leave this with 18, so that oeople who stil use 18 can use it as well

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we use jlink to compile the jdk with JabRef for distribution it does not matter for the end-user.
For the developer the sourceCompatibility is just a safetynet for not using newer features. The targetCompatibility is the JDK we are using. But we want to be able to use the latest JDK with all the latest security patches. Alternative would be to switch back to 17 LTS, but not to stay stuck on 18.
https://www.baeldung.com/gradle-sourcecompatiblity-vs-targetcompatibility

@Siedlerchr

Copy link
Copy Markdown
Member

Modernizer plugin fail refs andygoossens/gradle-modernizer-plugin#3

@ThiloteE ThiloteE added the dev: code-quality Issues related to code or architecture decisions label Dec 4, 2022
* upstream/main:
  Bump slf4j-api from 2.0.4 to 2.0.5 (#9428)
  Bump archunit-junit5-api from 1.0.0 to 1.0.1 (#9429)
  Bump jackson-datatype-jsr310 from 2.14.0 to 2.14.1 (#9430)
  Bump lucene-highlighter from 9.4.1 to 9.4.2 (#9431)
  Fix weird checkbox styling (#9425)
  New translations JabRef_en.properties (Italian) (#9424)
  New translations JabRef_en.properties (German) (#9418)
@Siedlerchr

This comment was marked as outdated.

@Siedlerchr Siedlerchr added the status: depends-on-external A bug or issue that depends on an update of an external library label Dec 12, 2022
koppor and others added 3 commits December 17, 2022 21:07
* upstream/main: (75 commits)
  Observable Preferences S (LastExportPath and Cleanups in JabRefPreferences and Globals) (#9493)
  Enable groups drag'n'drop to new library (#9460)
  Update MacOS jabrefHost.py to find local installs (#9487)
  Fix remember last open valid library with empty new one (#9489)
  Observable Preferences R (CitationKeyPatternPreferences) (#9486)
  Fixed ZBMathTest and extracted keyWordSeparator (#9485)
  New Crowdin updates (#9483)
  Add log for ignored excepton (#9302)
  Select Library to import into (#9402)
  Bump org.eclipse.jgit from 6.3.0.202209071007-r to 6.4.0.202211300538-r (#9476)
  Bump com.github.andygoossens.modernizer from 1.6.2 to 1.7.0 (#9478)
  Bump mockito-core from 4.9.0 to 4.10.0 (#9479)
  Bump checkstyle from 10.4 to 10.5.0 (#9477)
  Bump slf4j-api from 2.0.5 to 2.0.6 in /buildSrc (#9480)
  Bibtex month not deprecated (#9404)
  Show development information\n\n+semver: major
  Release v5.8
  Update external libraries add afterburner fx jabref add jakarta inject
  fix display name for artifact store
  Prepare CHANGELOG for release
  ...
@Siedlerchr Siedlerchr added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers and removed status: depends-on-external A bug or issue that depends on an update of an external library labels Dec 25, 2022
@Siedlerchr Siedlerchr merged commit 7fc7361 into main Dec 26, 2022
@Siedlerchr Siedlerchr deleted the update_jdk branch December 26, 2022 17:42
Siedlerchr added a commit that referenced this pull request Dec 27, 2022
Siedlerchr added a commit that referenced this pull request Dec 27, 2022
@wamserma wamserma mentioned this pull request Feb 8, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev: code-quality Issues related to code or architecture decisions 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.

Switch to Java 19

4 participants