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

Update to airbase 156 + other deps #1149

Merged
merged 3 commits into from May 6, 2024

Conversation

wendigo
Copy link
Collaborator

@wendigo wendigo commented Apr 29, 2024

No description provided.

@wendigo wendigo changed the title Fix checkstyle for array initialization Update to airbase 155 + other deps Apr 29, 2024
@wendigo
Copy link
Collaborator Author

wendigo commented Apr 29, 2024

cc @findepi

pom.xml Outdated Show resolved Hide resolved
@findepi
Copy link
Contributor

findepi commented Apr 29, 2024

LGTM % #1149 (comment).
let's update checkstyle in airbase (unless there is some reason for urgency to do otherwise)

@wendigo
Copy link
Collaborator Author

wendigo commented Apr 29, 2024

@findepi I can update this in airbase. Btw. when I've updated this in Trino - there are some previously unreported issues

@wendigo wendigo force-pushed the serafin/airbase-155 branch 2 times, most recently from c9f52af to 8441517 Compare April 29, 2024 11:15
pom.xml Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
@wendigo wendigo changed the title Update to airbase 155 + other deps Update to airbase 156 + other deps Apr 30, 2024
@wendigo
Copy link
Collaborator Author

wendigo commented Apr 30, 2024

AC @findepi @electrum

@wendigo
Copy link
Collaborator Author

wendigo commented Apr 30, 2024

@electrum please merge and release.

@wendigo
Copy link
Collaborator Author

wendigo commented May 3, 2024

Let's wait for Jetty 12.0.9 (we need it)

@electrum electrum merged commit 7c1ca29 into airlift:master May 6, 2024
3 checks passed
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