Skip to content

fix jbang#15649

Merged
Siedlerchr merged 1 commit into
mainfrom
fixjbang
Apr 30, 2026
Merged

fix jbang#15649
Siedlerchr merged 1 commit into
mainfrom
fixjbang

Conversation

@Siedlerchr

@Siedlerchr Siedlerchr commented Apr 30, 2026

Copy link
Copy Markdown
Member

Related issues and pull requests

Closes _____

PR Description

Steps to test

Checklist

  • I own the copyright of the code submitted and I license it under the MIT license
  • [.] I manually tested my changes in running JabRef (always required)
  • [.] I added JUnit tests for changes (if applicable)
  • [.] I added screenshots in the PR description (if change is visible to the user)
  • [.] I added a screenshot in the PR description showing a library with a single entry with me as author and as title the issue number
  • [.] I described the change in CHANGELOG.md in a way that can be understood by the average user (if change is visible to the user)
  • [.] I checked the user documentation for up to dateness and submitted a pull request to our user documentation repository

@Siedlerchr Siedlerchr added the automerge PR is tagged with that label will be merged if workflows are green label Apr 30, 2026
@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Update Grizzly framework dependencies to 5.0.1

✨ Enhancement

Grey Divider

Walkthroughs

Description 
• Update Grizzly framework dependencies to version 5.0.1
• Upgrades grizzly-framework and grizzly-http-server packages
Diagram 
flowchart LR
  A["JabSrvLauncher.java"] -- "Update Grizzly deps" --> B["grizzly-framework 5.0.1"]
  A -- "Update Grizzly deps" --> C["grizzly-http-server 5.0.1"]
Loading

Grey Divider

File Changes

1. .jbang/JabSrvLauncher.java Dependencies +2/-2

Upgrade Grizzly framework dependencies

• Updated org.glassfish.grizzly:grizzly-framework from 5.0.0 to 5.0.1
• Updated org.glassfish.grizzly:grizzly-http-server from 5.0.0 to 5.0.1

.jbang/JabSrvLauncher.java

Grey Divider

Qodo Logo

@Siedlerchr Siedlerchr added the dependencies Pull requests that update a dependency file label Apr 30, 2026
@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Apr 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

CI Feedback 🧐

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: Auto Merge

Failed stage: Merge PR [❌]

Failed test name: ""

Failure summary:

The action failed during the merge step because GitHub auto-merge is not permitted for this
repository:
- The workflow ran gh pr merge --auto --squash "$PR_URL" for
#15649.
- GitHub returned GraphQL: Auto merge is not allowed
for this repository (enablePullRequestAutoMerge), indicating auto-merge is disabled and/or merges
must go through the merge queue (The merge strategy for main is set by the merge queue).
- As a
result, the gh pr merge command exited with code 1, causing the action to fail.

Relevant error logs: 
1:  ##[group]Runner Image Provisioner
2:  Hosted Compute Agent
...

38:  �[36;1mgh pr review --approve "$PR_URL"�[0m
39:  shell: /usr/bin/bash -e {0}
40:  env:
41:  PR_URL: https://github.com/JabRef/jabref/pull/15649
42:  GITHUB_TOKEN: ***
43:  ##[endgroup]
44:  ##[group]Run gh pr merge --auto --squash "$PR_URL"
45:  �[36;1mgh pr merge --auto --squash "$PR_URL"�[0m
46:  shell: /usr/bin/bash -e {0}
47:  env:
48:  PR_URL: https://github.com/JabRef/jabref/pull/15649
49:  GITHUB_TOKEN: ***
50:  ##[endgroup]
51:  ! The merge strategy for main is set by the merge queue
52:  GraphQL: Auto merge is not allowed for this repository (enablePullRequestAutoMerge)
53:  ##[error]Process completed with exit code 1.
54:  Cleaning up orphan processes

@Siedlerchr Siedlerchr enabled auto-merge April 30, 2026 17:40
@Siedlerchr Siedlerchr disabled auto-merge April 30, 2026 17:52
@Siedlerchr Siedlerchr merged commit 6d32407 into main Apr 30, 2026
69 of 87 checks passed
@Siedlerchr Siedlerchr deleted the fixjbang branch April 30, 2026 17:52
Siedlerchr added a commit that referenced this pull request May 1, 2026
@Siedlerchr
Merge remote-tracking branch 'upstream/fixSecurityIssues' into fixSec…
09c35b9 
…urityIssues

* upstream/fixSecurityIssues:
  fix jbang (#15649)
  Add initial claude action support
  Chore(deps): Bump org.glassfish.grizzly:grizzly-bom in /versions (#15648)
  Update IntelliJ settings (#15629)
  Chore(deps): Bump org.openrewrite.recipe:rewrite-recipe-bom (#15626)
  Fixed year not appearing on citation key for In-* entries due to orphaned crossref links (#9071) (#15582)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jabref-machine jabref-machine jabref-machine approved these changes

Assignees

No one assigned

Labels

automerge PR is tagged with that label will be merged if workflows are green dependencies Pull requests that update a dependency file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Siedlerchr @jabref-machine