Skip to content

[Git integration] Add proper closing at testing#13626

Merged
wanling0000 merged 1 commit into
mainfrom
add-closing
Aug 1, 2025
Merged

[Git integration] Add proper closing at testing#13626
wanling0000 merged 1 commit into
mainfrom
add-closing

Conversation

@koppor

@koppor koppor commented Aug 1, 2025

Copy link
Copy Markdown
Member

Follow-up to #13518

Adds some fixes because of eclipse-jgit/jgit#155 (comment)

Tetss now run on Windows again.

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.

.setInitialBranch("main")
.call();
.call()) {
// This ensures the remote repository is initialized and properly closed

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Comment is trivial and only restates what is obvious from the try-with-resources block. The comment does not provide additional information or reasoning.

@subhramit subhramit changed the title Add proper closing at testing [Git integration] Add proper closing at testing Aug 1, 2025
@subhramit subhramit added component: git project: gsoc dev: hotfix High-level fix for a bug, may need root-cause investigation in future. labels Aug 1, 2025
@subhramit subhramit enabled auto-merge August 1, 2025 19:14
@subhramit subhramit added this pull request to the merge queue Aug 1, 2025
@subhramit subhramit removed this pull request from the merge queue due to a manual request Aug 1, 2025
@wanling0000 wanling0000 added this pull request to the merge queue Aug 1, 2025
Merged via the queue into main with commit 17d4492 Aug 1, 2025
2 checks passed
@wanling0000 wanling0000 deleted the add-closing branch August 1, 2025 21:16
Siedlerchr added a commit that referenced this pull request Aug 2, 2025
* upstream/main:
  Issue 13619 Make Citation relations text more clear. (#13620)
  Explain how to handle notifications (#13630)
  Fix scope of 'determine issue number' job (#13627)
  Add proper closing (#13626)
  Implement logic orchestration for Git Pull/Push operations (#13518)
  Make pattern for issue number more strict
  Fix "Cannot load file MultiMergeEntries.fxml" (#13624)
  Add Copy markdown to copy citation (#13387)
  Add ADR-0047 (#13621)
  Initial start of implementing a LSP for integrity checks (#13612)
  Refactor merge entries package structure (#13614)
  New Crowdin updates (#13616)
  BibEntry class no longer implements Cloneable (#13615)
  Fix dark mode in {} of Citation Relations tab of the entry editor (#13609)
  Update dependency org.kohsuke:github-api to v2.0-rc.4 (#13611)
  Fix setting of proxy without password (#13605)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: git dev: hotfix High-level fix for a bug, may need root-cause investigation in future. project: gsoc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants