Skip to content

Conversation

@koppor
Copy link
Member

@koppor koppor commented Oct 11, 2024

Follow-up to #11729.

I needed to remove --automaticallySetFileLinks, because this needs a data structure pointing to an icon. Solving that will take some hours. To move forward with the multi module build, I thought its best to remove it (for know), introduce the multimodule project setup afterwards and then try to get it running again.

Mandatory checks

  • 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.

github-actions[bot]

This comment was marked as outdated.

github-actions[bot]

This comment was marked as outdated.

github-actions[bot]

This comment was marked as outdated.

@github-actions
Copy link
Contributor

The build of this PR is available at https://builds.jabref.org/pull/11921/merge.

calixtus
calixtus previously approved these changes Oct 14, 2024
@calixtus calixtus enabled auto-merge October 14, 2024 20:55
@github-actions
Copy link
Contributor

The build of this PR is available at https://builds.jabref.org/pull/11921/merge.

…uild

# Conflicts:
#	src/main/java/org/jabref/Launcher.java
@calixtus calixtus added this pull request to the merge queue Oct 14, 2024
@github-actions
Copy link
Contributor

The build of this PR is available at https://builds.jabref.org/pull/11921/merge.

Merged via the queue into main with commit 96ea003 Oct 14, 2024
1 check passed
@calixtus calixtus deleted the prepare-multimodule-build branch October 14, 2024 21:41
@github-actions
Copy link
Contributor

The build for this PR is no longer available. Please visit https://builds.jabref.org/main/ for the latest build.

@koppor koppor mentioned this pull request Apr 24, 2025
23 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants