-
Notifications
You must be signed in to change notification settings - Fork 320
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
Issue 2235 - Update to Jetty 11 #2758
Conversation
Signed-off-by: Florian Pirchner <florian.pirchner@gmail.com>
Signed-off-by: Florian Pirchner <florian.pirchner@gmail.com>
Signed-off-by: Florian Pirchner <florian.pirchner@gmail.com>
thx @florianpirchner |
Signed-off-by: Florian Pirchner <florian.pirchner@gmail.com>
@cdietrich - i am not sure, why continuous-integration/jenkins/pr-head cannot be built. Found no further information in build, accept: But i did not change anything in the target definition. Best, Flo |
There were changes in orbit. You need to rebase on this mornings change on main branch |
Signed-off-by: Florian Pirchner <florian.pirchner@gmail.com>
@florianpirchner once the build is green, can you rebase/squash etc this pr so that it consists of a single commit. |
I rebased the commits in a new branch. Is there a way to change the branch in this PR? Or should i create new PR? Sorry, my first time using PR and google says many different things about best practice. Best Florian |
Guess you have to force push on this branch. Or simply open new pr |
@cdietrich hi christian, new PR is #2763. Hope i did everything fine :) Best Flo |
Hi Christian,
i signed the contributor agreement and happy to provide you with this pull request.
Best, Flo