Skip to content

WIP convert to jackson 3 #14259

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

WIP convert to jackson 3 #14259

wants to merge 30 commits into from
+64 −41

Conversation

@Siedlerchr
Copy link
Member

@Siedlerchr Siedlerchr commented Nov 8, 2025

Closes https://github.com/JabRef/jabref-issue-melting-pot/issues/1083

Steps to test

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • [.] I manually tested my changes in running JabRef (always required)
  • [.] I added JUnit tests for changes (if applicable)
  • [.] I added screenshots in the PR description (if change is visible to the user)
  • [.] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • [.] I checked the user 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 updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

@Siedlerchr
Copy link
Member Author

Siedlerchr commented Nov 9, 2025

stack overflow error in jbang wtf?

@Siedlerchr
Merge branch 'main' into jackson3
2b33910 
* main:
  change command imports
  Chore(deps): Bump org.openrewrite.rewrite from 7.19.0 to 7.20.0 (#14266)
  Chore(deps): Bump org.openrewrite.recipe:rewrite-recipe-bom from 3.17.0 to 3.18.0 (#14265)
  Chore(deps): Bump com.google.errorprone:error_prone_core in /versions (#14269)
  Chore(deps): Bump io.github.thibaultmeyer:cuid in /versions (#14268)
  Chore(deps): Bump org.gradlex:java-module-testing in /build-logic (#14267)
  Chore(deps): Bump jablib/src/main/resources/csl-styles (#14264)
  Chore(deps): Bump appleboy/ssh-action from 1.2.2 to 1.2.3 (#14262)
  Fix: Ctrl+Shift+L now opens terminal in the active library directory (#14256)

# Conflicts:
#	jabsrv/src/main/java/module-info.java
#	jabsrv/src/main/java/org/jabref/http/server/command/CommandResource.java
@Siedlerchr
fix imports
431c9e9
@Siedlerchr
fix imports
9b30927
@koppor koppor mentioned this pull request Nov 9, 2025
Merged
3 tasks
@Siedlerchr
Copy link
Member Author

Siedlerchr commented Nov 9, 2025

definitely something with the dep:
at dev.jbang.dependencies.DependencyUtil.looksLikeAPossibleGav(DependencyUtil.java:158)
at dev.jbang.source.parser.Directives.isGav(Directives.java:82)
at java.base/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:196)

@koppor
Copy link
Member

koppor commented Nov 9, 2025

I filed jbangdev/jbang#2281

@Siedlerchr
fix imports
f291512
@Siedlerchr
add jackson deps to jbang
395fade
@Siedlerchr
fix import
ee56c1e
@Siedlerchr
Merge branch 'main' into jackson3
bc7b308 
* main:
  Fix calling JBang command (#14282)
  .jbang scripts should not be included inside themselves (#14278)
  Fix typos (#14277)
  Refactor importDatabase for better readability (#14274)
  fix jbang (#14276)
  Feat: Add definition links for TeX Files (#14260)
  Update preventing module commits (#14273)
  Fix JabKitLuancher
  Rename packages in jabkit to start with org.jabref.toolkit to avoid split packages between jabkit and jabgui (#14052). (#14271)
  Add doi-to-bibtex to examples and JabKit (#14244)
@Siedlerchr
fix imports
bf6b1dc
@Siedlerchr
reveRT CSL CHANGE
d7b6e80
@Siedlerchr
fix imports
b828262
@Siedlerchr
add to example
a1ab067
@Siedlerchr
cahnge mscocde utils
b4aaae4
@Siedlerchr
Copy link
Member Author

Siedlerchr commented Nov 11, 2025

needs testing with LSP

@Siedlerchr
Merge remote-tracking branch 'upstream/main' into jackson3
f404793 
* upstream/main:
  Fix NPE on startup (#14272)
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into jackson3
5aee419 
* upstream/main: (30 commits)
  Chore(deps): Bump io.github.classgraph:classgraph from 4.8.181 to 4.8.184 in /versions (#14304)
  Chore(deps): Bump com.fasterxml:aalto-xml in /versions (#14311)
  Chore(deps): Bump commons-io:commons-io in /versions (#14310)
  Chore(deps): Bump org.apache.maven.plugins:maven-surefire-plugin (#14298)
  Disable fetcher-gui-test (#14308)
  Chore(deps): Bump com.dlsc.gemsfx:gemsfx in /versions (#14307)
  No labels for dependeabot updates
  Fix fallback window height from 786 to 768 (#14295)
  Chore(deps): Bump com.fasterxml.jackson.dataformat:jackson-dataformat-yaml (#14306)
  Chore(deps): Bump org.apache.maven.plugins:maven-compiler-plugin (#14302)
  Chore(deps): Bump org.apache.maven.plugins:maven-deploy-plugin (#14300)
  Chore(deps): Bump org.apache.maven.plugins:maven-clean-plugin (#14299)
  Chore(deps): Bump jablib/src/main/resources/csl-styles (#14296)
  Chore(deps): Bump com.vanniktech.maven.publish in /jablib (#14303)
  Chore(deps): Bump org.apache.maven.plugins:maven-project-info-reports-plugin (#14297)
  Update all dependencies (#14301)
  Prepare maven3 example project (#14294)
  Refactor StringUtilTest to use parameterized tests (#14126)
  Try to fix download of pr_number
  Fix workflow names
  ...
@Siedlerchr
fix lsp import
fcfebab
@Siedlerchr
ad jackson annotaitons
5891f4b
@Siedlerchr
ad jackson annotaitons
00f1f31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Siedlerchr @koppor