Skip to content

Conversation

@koppor
Copy link
Member

@koppor koppor commented May 23, 2025

Required for JabRef/user-documentation#569

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.

@koppor koppor added the automerge PR is tagged with that label will be merged if workflows are green label May 23, 2025
@koppor koppor enabled auto-merge May 23, 2025 12:30
//DEPS org.eclipse.jgit:org.eclipse.jgit.pgm:7.2.1.202505142326-r

public class CloneJabRef {
public static void main(String[] noArgs) throws Exception {
Copy link

Choose a reason for hiding this comment

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

Using 'throws Exception' is too broad and not specific. It is better to specify the exact exceptions that might be thrown to improve code readability and maintainability.

@koppor koppor disabled auto-merge May 23, 2025 12:34
@koppor koppor merged commit 9c33a08 into main May 23, 2025
2 checks passed
@koppor koppor deleted the add-clone-jabref branch May 23, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge PR is tagged with that label will be merged if workflows are green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants