Skip to content

Commit df0a512

Browse files
committed
Merge remote-tracking branch 'origin/LSP-add-features' into LSP-add-features
2 parents 950c556 + c21b8a8 commit df0a512

File tree

78 files changed

+3335
-627
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

78 files changed

+3335
-627
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,8 @@ Keep ALL the items. Replace the dots inside [.] and mark them as follows:
2525
-->
2626

2727
- [x] I own the copyright of the code submitted and I license it under the [MIT license](https://github.com/JabRef/jabref/blob/main/LICENSE)
28-
- [.] Change in `CHANGELOG.md` described in a way that is understandable for the average user (if change is visible to the user)
29-
- [.] Tests created for changes (if applicable)
30-
- [.] Manually tested changed features in running JabRef (always required)
31-
- [.] Screenshots added in PR description (if change is visible to the user)
32-
- [.] [Checked developer's documentation](https://devdocs.jabref.org/): Is the information available and up to date? If not, I outlined it in this pull request.
33-
- [.] [Checked documentation](https://docs.jabref.org/): 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.
28+
- [.] I manually tested my changes in running JabRef (always required)
29+
- [.] I added JUnit tests for changes (if applicable)
30+
- [.] I added screenshots in the PR description (if change is visible to the user)
31+
- [.] I described the change in `CHANGELOG.md` in a way that is understandable for the average user (if change is visible to the user)
32+
- [.] I checked the [user documentation](https://docs.jabref.org/): 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>.

.github/ghprcomment.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,10 @@
130130
Hey, we noticed that you **force-pushed** your changes.
131131
Force pushing is a bad practice when working together on a project (mainly because it is [not supported well by GitHub itself](https://github.com/orgs/community/discussions/3478)).
132132
Commits are lost and comments on commits lose their context, thus making it harder to review changes.
133-
At the end, all commits will be [squashed](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/about-pull-request-merges#squash-and-merge-your-commits) anyway before being merged into the `main` branch.
133+
134+
135+
When the pull request is getting integrated into `main`, all commits will be [squashed](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/about-pull-request-merges#squash-and-merge-your-commits) anyway.
136+
Thus, your individual commit history will not be visible in `main`.
134137
135138
136139
In future, **please avoid that**. For now, you can continue working.

.github/workflows/binaries.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ jobs:
193193
id: gitversion
194194
uses: gittools/actions/gitversion/[email protected]
195195
- name: Setup JDK
196-
uses: actions/setup-java@v4
196+
uses: actions/setup-java@v5
197197
with:
198198
java-version: '24'
199199
distribution: 'corretto'

.github/workflows/on-issue-comment.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,10 @@ jobs:
3333
3434
Happy coding! 🚀
3535
assignment_suggestion_comment: >
36-
👋 Hey @{{ handle }}, looks like you’re eager to work on this issue—great! 🎉
37-
It also looks like you skipped reading our [CONTRIBUTING.md](https://github.com/JabRef/jabref/blob/main/CONTRIBUTING.md), which explains exactly how to participate. No worries, it happens to the best of us.
38-
Give it a read, and you’ll discover the ancient wisdom of assigning issues to yourself. Trust me, it’s worth it. 🚀
36+
👋 Hey @{{ handle }}, looks like you’re eager to work on this issue — great! 🎉
37+
It also looks like you don't know how to assign issues to yourself.
38+
Please read our [CONTRIBUTING.md](https://github.com/JabRef/jabref/blob/main/CONTRIBUTING.md) to find out how.
39+
It will also guide you in other aspects of contributing to JabRef.
3940
- name: Move issue corresponding column in "Candidates for University Projects"
4041
uses: m7kvqbe1/github-action-move-issues@main
4142
if: ${{ steps.assign.outputs.assigned == 'yes' || steps.assign.outputs.unassigned == 'yes' }}

.github/workflows/publish.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ jobs:
5353
id: gitversion
5454
uses: gittools/actions/gitversion/[email protected]
5555
- name: Setup JDK
56-
uses: actions/setup-java@v4
56+
uses: actions/setup-java@v5
5757
with:
5858
java-version: '24'
5959
distribution: 'corretto'

.github/workflows/run-openrewrite.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,10 @@ jobs:
3131
GH_TOKEN: ${{secrets.GH_TOKEN_JABREF_MACHINE_PR_APPROVE}}
3232

3333
- name: Set up JDK
34-
uses: actions/setup-java@v4
34+
uses: actions/setup-java@v5
3535
with:
36-
java-version: 24.0.1
37-
distribution: 'zulu'
36+
java-version: 24
37+
distribution: 'corretto'
3838
- name: Setup Gradle
3939
uses: gradle/actions/setup-gradle@v4
4040
- name: Generate JBang cache key

.github/workflows/tests-code-fetchers.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,10 @@ jobs:
5353
submodules: 'true'
5454
show-progress: 'false'
5555
- name: Set up JDK
56-
uses: actions/setup-java@v4
56+
uses: actions/setup-java@v5
5757
with:
58-
java-version: 24.0.1
59-
distribution: 'zulu'
58+
java-version: 24
59+
distribution: 'corretto'
6060
- name: Setup Gradle
6161
uses: gradle/actions/setup-gradle@v4
6262
- name: Run fetcher tests

.github/workflows/tests-code.yml

Lines changed: 30 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,10 @@ jobs:
4545
submodules: 'true'
4646
show-progress: 'false'
4747
- name: Set up JDK
48-
uses: actions/setup-java@v4
48+
uses: actions/setup-java@v5
4949
with:
50-
java-version: 24.0.1
51-
distribution: 'zulu'
50+
java-version: 24
51+
distribution: 'corretto'
5252
- name: Run checkstyle reporter
5353
uses: dbelyaev/action-checkstyle@master
5454
with:
@@ -84,10 +84,10 @@ jobs:
8484
submodules: 'true'
8585
show-progress: 'false'
8686
- name: Set up JDK
87-
uses: actions/setup-java@v4
87+
uses: actions/setup-java@v5
8888
with:
89-
java-version: 24.0.1
90-
distribution: 'zulu'
89+
java-version: 24
90+
distribution: 'corretto'
9191
- name: Setup Gradle
9292
uses: gradle/actions/setup-gradle@v4
9393
- name: Generate JBang cache key
@@ -118,10 +118,10 @@ jobs:
118118
submodules: 'true'
119119
show-progress: 'false'
120120
- name: Set up JDK
121-
uses: actions/setup-java@v4
121+
uses: actions/setup-java@v5
122122
with:
123-
java-version: 24.0.1
124-
distribution: 'zulu'
123+
java-version: 24
124+
distribution: 'corretto'
125125
- name: Setup Gradle
126126
uses: gradle/actions/setup-gradle@v4
127127
- name: Generate JBang cache key
@@ -235,10 +235,10 @@ jobs:
235235
- name: Checkout source
236236
uses: actions/checkout@v5
237237
- name: Set up JDK
238-
uses: actions/setup-java@v4
238+
uses: actions/setup-java@v5
239239
with:
240-
java-version: 24.0.1
241-
distribution: 'zulu'
240+
java-version: 24
241+
distribution: 'corretto'
242242
- name: Setup Gradle
243243
uses: gradle/actions/setup-gradle@v4
244244
- name: Generate JBang cache key
@@ -270,10 +270,10 @@ jobs:
270270
submodules: 'true'
271271
show-progress: 'false'
272272
- name: Set up JDK
273-
uses: actions/setup-java@v4
273+
uses: actions/setup-java@v5
274274
with:
275-
java-version: 24.0.1
276-
distribution: 'zulu'
275+
java-version: 24
276+
distribution: 'corretto'
277277
- name: Generate JBang cache key
278278
id: cache-key
279279
shell: bash
@@ -318,10 +318,10 @@ jobs:
318318
submodules: 'true'
319319
show-progress: 'false'
320320
- name: Set up JDK
321-
uses: actions/setup-java@v4
321+
uses: actions/setup-java@v5
322322
with:
323-
java-version: 24.0.1
324-
distribution: 'zulu'
323+
java-version: 24
324+
distribution: 'corretto'
325325
- name: Setup Gradle
326326
uses: gradle/actions/setup-gradle@v4
327327
- name: Generate JBang cache key
@@ -367,10 +367,10 @@ jobs:
367367
submodules: 'true'
368368
show-progress: 'false'
369369
- name: Set up JDK
370-
uses: actions/setup-java@v4
370+
uses: actions/setup-java@v5
371371
with:
372-
java-version: 24.0.1
373-
distribution: 'zulu'
372+
java-version: 24
373+
distribution: 'corretto'
374374
- name: Setup Gradle
375375
uses: gradle/actions/setup-gradle@v4
376376
- name: Generate JBang cache key
@@ -405,10 +405,10 @@ jobs:
405405
submodules: 'false'
406406
show-progress: 'false'
407407
- name: Set up JDK
408-
uses: actions/setup-java@v4
408+
uses: actions/setup-java@v5
409409
with:
410-
java-version: 24.0.1
411-
distribution: 'zulu'
410+
java-version: 24
411+
distribution: 'corretto'
412412
- name: Generate JBang cache key
413413
id: cache-key
414414
shell: bash
@@ -466,10 +466,10 @@ jobs:
466466
show-progress: 'false'
467467
- name: Set up JDK
468468
if: github.ref == 'refs/heads/main'
469-
uses: actions/setup-java@v4
469+
uses: actions/setup-java@v5
470470
with:
471-
java-version: 24.0.1
472-
distribution: 'zulu'
471+
java-version: 24
472+
distribution: 'corretto'
473473
- name: Setup Gradle
474474
uses: gradle/actions/setup-gradle@v4
475475
- name: Generate JBang cache key
@@ -511,10 +511,10 @@ jobs:
511511
with:
512512
show-progress: 'false'
513513
- name: Set up JDK
514-
uses: actions/setup-java@v4
514+
uses: actions/setup-java@v5
515515
with:
516-
java-version: 24.0.1
517-
distribution: 'zulu'
516+
java-version: 24
517+
distribution: 'corretto'
518518
- name: Setup Gradle
519519
uses: gradle/actions/setup-gradle@v4
520520
- run: ./gradlew traceRequirements

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ Note that this project **does not** adhere to [Semantic Versioning](https://semv
5757
- We added markdown rendering and copy capabilities to AI chat responses. [#12234](https://github.com/JabRef/jabref/issues/12234)
5858
- We added a new `jabkit` command `pseudonymize` to pseudonymize the library. [#13109](https://github.com/JabRef/jabref/issues/13109)
5959
- We added functionality to focus running instance when trying to start a second instance. [#13129](https://github.com/JabRef/jabref/issues/13129)
60+
- We added a "Copy Field Content" submenu to the entry context menu, allowing users to quickly copy specific field contents including Author, Journal, Date, Keywords, and Abstract fields from selected entries. [#13280](https://github.com/JabRef/jabref/pull/13280)
6061
- We added a highlighted diff regarding changes to the Group Tree Structure of a bib file, made outside JabRef. [#11221](https://github.com/JabRef/jabref/issues/11221)
6162
- We added a new setting in the 'Entry Editor' preferences to hide the 'File Annotations' tab when no annotations are available. [#13143](https://github.com/JabRef/jabref/issues/13143)
6263
- We added support for multi-file import across different formats. [#13269](https://github.com/JabRef/jabref/issues/13269)
@@ -72,8 +73,11 @@ Note that this project **does not** adhere to [Semantic Versioning](https://semv
7273
- We added a progress dialog to the "Check consistency" action and progress output to the corresponding cli command. [#12487](https://github.com/JabRef/jabref/issues/12487)
7374
- We made the `check-consistency` command of the toolkit always return an exit code; 0 means no issues found, a non-zero exit code reflects any issues, which allows CI to fail in these cases [#13328](https://github.com/JabRef/jabref/issues/13328).
7475
- We changed the validation error dialog for overriding the default file directories to a confirmation dialog for saving other preferences under the library properties. [#13488](https://github.com/JabRef/jabref/pull/13488)
76+
- We made the copy sub menu on the context menu consistent with the copy sub menu at "Edit". [#13280](https://github.com/JabRef/jabref/pull/13280)
7577
- We improved file exists warning dialog with clearer options and tooltips [#12565](https://github.com/JabRef/jabref/issues/12565)
7678
- We introduced walkthrough functionality [#12664](https://github.com/JabRef/jabref/issues/12664)
79+
- The Welcome tab now has a responsive layout. [#12664](https://github.com/JabRef/jabref/issues/12664)
80+
- We introduced a donation prompt in the Welcome tab. [#12664](https://github.com/JabRef/jabref/issues/12664)
7781

7882
### Fixed
7983

@@ -96,12 +100,14 @@ Note that this project **does not** adhere to [Semantic Versioning](https://semv
96100
- We excluded specific fields (e.g., `comment`, `pdf`, `sortkey`) from the consistency check to reduce false positives [#13131](https://github.com/JabRef/jabref/issues/13131)
97101
- We fixed an issue where moved or renamed linked files in the file directory were not automatically relinked by the “search for unlinked files” feature. [#13264](https://github.com/JabRef/jabref/issues/13264)
98102
- We fixed an issue with proxy setup in the absence of a password. [#12412](https://github.com/JabRef/jabref/issues/12412)
103+
- We fixed an issue with the targets of the menu item "copy to". [#13741](https://github.com/JabRef/jabref/pull/13741)
99104
- We fixed an issue where the tab showing the fulltext search results was not displayed. [#12865](https://github.com/JabRef/jabref/issues/12865)
100105
- We fixed an issue showing an empty tooltip in maintable. [#11681](https://github.com/JabRef/jabref/issues/11681)
101106
- We fixed an issue displaying a warning if a file to open is not found. [#13430](https://github.com/JabRef/jabref/pull/13430)
102107
- We fixed an issue where Document Viewer showed technical exceptions when opening entries with non-PDF files. [#13198](https://github.com/JabRef/jabref/issues/13198)
103108
- When creating a library, if you drag a PDF file containing only a single column, the dialog will now automatically close. [#13262](https://github.com/JabRef/jabref/issues/13262)
104109
- We fixed an issue where the tab showing the fulltext search results would appear blank after switching library. [#13241](https://github.com/JabRef/jabref/issues/13241)
110+
- We fixed an issue where "Copy to" was enabled even if no other library was opened. [#13280](https://github.com/JabRef/jabref/pull/13280)
105111
- We fixed an issue where the groups were still displayed after closing all libraries. [#13382](https://github.com/JabRef/jabref/issues/13382)
106112
- Enhanced field selection logic in the Merge Entries dialog when fetching from DOI to prefer valid years and entry types. [#12549](https://github.com/JabRef/jabref/issues/12549)
107113

CONTRIBUTING.md

Lines changed: 20 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -97,27 +97,30 @@ GitHub will then automatically assign you.
9797

9898
1. Follow the steps at [Pre Condition 3: Code on the local machine](https://devdocs.jabref.org/getting-into-the-code/guidelines-for-setting-up-a-local-workspace/pre-03-code) to a) create a fork and b) have the fork checked out on your local machine
9999
2. Ensure that you followed the [steps to set up a local workspace](https://devdocs.jabref.org/getting-into-the-code/guidelines-for-setting-up-a-local-workspace/) to have the code running properly in IntelliJ.
100-
3. **Create a new branch** (such as `fix-for-issue-121`). Be sure to create a **separate branch** for each improvement you implement.
101-
4. Read our [architecture and components](https://devdocs.jabref.org/architecture-and-components).
102-
5. Work on the **new branch — not the `main` branch.** Refer to our [code how-tos](https://devdocs.jabref.org/code-howtos) if you have questions about your implementation.
103-
6. Create a [pull request to JabRef main repository](https://github.com/JabRef/jabref/pulls).
104-
For an overview on the concept of pull requests, take a look at GitHub's [pull request help documentation](https://help.github.com/articles/about-pull-requests/).
105-
1. Ensure that you followed the requirements listed below. They are not too hard, they merely support the maintainers to focus on supportive feedback than just stating the obvious.
106-
2. For text inspirations, consider [How to write the perfect pull request](https://github.com/blog/1943-how-to-write-the-perfect-pull-request).
107-
3. In case your pull request is not yet complete or not yet ready for review, create a [draft pull request](https://github.blog/2019-02-14-introducing-draft-pull-requests/) instead.
108-
7. Wait for automatic checks to run and bots commenting.
109-
8. Address the feedback of the automated checks. To find solutions to the most common errors that lead to such failures, check our [FAQ page](https://devdocs.jabref.org/code-howtos/faq).
110-
9. Wait for feedback of one of the developers.
111-
10. Address the feedback of the developer.
112-
11. Wait for feedback of a second developer.
113-
12. Address the feedback of the second developer.
114-
13. After two developers gave their green flag, the pull request will be merged.
100+
3. Read about our [architecture and components](https://devdocs.jabref.org/architecture-and-components).
101+
4. **Create a new branch** (such as `fix-for-issue-121`). Be sure to create a **separate branch** for each improvement you implement.
102+
5. Refer to our [code how-tos](https://devdocs.jabref.org/code-howtos) if you have questions about your implementation.
103+
6. Implement and test your changes.
104+
* Create JUnit tests for your changes, apart from manual testing. Maybe even use [Test-driven Development](https://en.wikipedia.org/wiki/Test-driven_development) to speed up your development.
105+
* Have fun. Learn. Communictate.
106+
7. Create a [pull request to JabRef main repository](https://github.com/JabRef/jabref/pulls).
107+
* For an overview on the concept of pull requests, take a look at GitHub's [pull request help documentation](https://help.github.com/articles/about-pull-requests/).
108+
* For text inspirations, consider [How to write the perfect pull request](https://github.com/blog/1943-how-to-write-the-perfect-pull-request).
109+
* Note that submission of a pull request takes time. There is also a [checklist](https://github.com/JabRef/jabref/blob/main/.github/PULL_REQUEST_TEMPLATE.md?plain=1), which takes time to check - and maybe update your branch.
110+
* In case your pull request is not yet complete or not yet ready for review, create a [draft pull request](https://github.blog/2019-02-14-introducing-draft-pull-requests/) instead.
111+
8. Wait for automatic checks to run and bots commenting.
112+
9. Address the feedback of the automated checks. To find solutions to the most common errors that lead to such failures, check our [FAQ page](https://devdocs.jabref.org/code-howtos/faq).
113+
10. Wait for feedback of one of the maintainers. Since this is a hobby project for all of them, this may take a while. You can speed up things by reducing their load. You can, for instance, try out other pull requests, review other pull requests, and answer questions on JabRef in the forum.
114+
11. Address the feedback of the maintainer. In case your pull request changed something significant, we might ask you to create or update [`docs/requirements`](https://devdocs.jabref.org/requirements/) to include a description of the requirement stemming from the issue - and link that with your implementation.
115+
12. Wait for feedback of a second maintainer.
116+
13. Address the feedback of the second maintainer.
117+
14. After two maintainers gave their green flag, the pull request will be merged.
115118

116119
We view pull requests as a collaborative process.
117120
Submit a pull request early to enable feedback from the team while you continue working.
118-
Please also remember to discuss bigger changes early with the core developers to ensure properly spend time and work.
121+
Please also remember to discuss bigger changes early with the core maintainers to ensure more fruitful investment of time and work, and lesser friction in acceptance later.
119122
Some fundamental design decisions can be found within our list of [Architectural Decision Records](https://devdocs.jabref.org/decisions/).
120-
After a pull request is ready for review, we will discuss improvements with you and agree to merge them once the [developers](https://github.com/JabRef/jabref/blob/main/MAINTAINERS) approve.
123+
After a pull request is ready for review, we will discuss improvements with you and agree to merge them once the [maintainers](https://github.com/JabRef/jabref/blob/main/MAINTAINERS) approve.
121124

122125
In case you have any questions, please
123126

0 commit comments

Comments
 (0)