Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade to jetty 12 #42

Closed

Conversation

andrew-tram
Copy link

upgrade to jetty 12

#41

Thank you for contributing to JGit!

JGit uses GerritHub for code changes and review, therefore pull requests in this repository cannot be merged.

Please make sure you have read the section on contributing patches of the Contributor Guide.

upgrade to jetty 12
@andrew-tram
Copy link
Author

After reading through the contributor guide, I'm still unable to submit changes for review. Unfortunately, I won't be able to commit any more time to this and will leave it as it is.

@davido
Copy link
Contributor

davido commented May 4, 2024

Unfortunately, I won't be able to commit any more time to this and will leave it as it is.

Can you provide more insight, what exactly doesn't work for you, or mention the error message during the push?

@davido
Copy link
Contributor

davido commented May 4, 2024

@lucamilanesio I tried to import this PR to gerrithub using github-plugin but got an error during the import. Can you clarify, if the import workflow of PRs is supported for JGit project?

@davido
Copy link
Contributor

davido commented May 4, 2024

I cherry-picked the PR and uploaded it to Gerrithub:

https://review.gerrithub.io/c/eclipse-jgit/jgit/+/1194275

@msohn
Copy link
Member

msohn commented Jun 9, 2024

Update to jetty 12 was submitted on the next branch
https://eclipse.gerrithub.io/c/eclipse-jgit/jgit/+/1194291
and will be merged to master soon
https://eclipse.gerrithub.io/c/eclipse-jgit/jgit/+/1195683

@msohn msohn closed this Jun 9, 2024
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.

None yet

3 participants