Skip to content

Conversation

@jjohannes
Copy link
Collaborator

Closes #13509

Steps to test

This is a follow up to #13511 that also reverts the names of "unnamed" modules not used in jablib but in other components. I think this is good to have consistency in this topic.

This was done using a new (not yet released) check in the extra-java-module-info plugin:
gradlex-org/extra-java-module-info#189

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • [ ] Change in CHANGELOG.md described in a way that is understandable for the average user (if change is visible to the user)
  • [ ] Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • [ ] Screenshots added in PR description (if change is visible to the user)
  • 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.

@trag-bot
Copy link

trag-bot bot commented Jul 9, 2025

@trag-bot didn't find any issues in the code! ✅✨

@jjohannes jjohannes requested a review from koppor July 9, 2025 08:34
@calixtus calixtus added dev: build-system dev: no-bot-comments If set, there should be no comments from our bots labels Jul 9, 2025
@koppor koppor enabled auto-merge July 9, 2025 08:54
@koppor koppor added this pull request to the merge queue Jul 9, 2025
Merged via the queue into main with commit 71c230f Jul 9, 2025
2 checks passed
@koppor koppor deleted the module-name-revert-2 branch July 9, 2025 09:15
@github-actions github-actions bot mentioned this pull request Jul 9, 2025
Siedlerchr added a commit that referenced this pull request Aug 2, 2025
* upstream/main:
  Also label PR if good first issue is made (#13526)
  New Crowdin updates (#13529)
  chore(deps): update dependency org.apache.logging.log4j:log4j-to-slf4j to v2.25.1 (#13528)
  chore: bump-okhttp-4.12.0-to-5.0.0 (#13521)
  Have the picker always on top (#13525)
  Refactor PushToApplications and split into logic and GUI (#13514)
  Add field to change HTTP port (#13479)
  Fix trigger of comment
  Use Java 11 isBlank (#13523)
  Add run openrewrite (#13524)
  Comment ion PR just opened (#13522)
  Improve merge logic to prefer valid year and entry type (#13506)
  Update dependency com.konghq:unirest-modules-gson to v4.4.12 (#13517)
  Add rpm target (#13516)
  Revert module name changes for remaining 'unnamed' Jars (#13515)
  fix: revert Java module names to restore Status Log compatibility in JabRef 5.15 (#13511)
  update java vendor in devcontainer and sdkmanrc (#13513)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev: build-system dev: no-bot-comments If set, there should be no comments from our bots

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change module names

4 participants