Re: [PR] Bump logback.version from 1.5.3 to 1.5.4 [tika]

2024-04-10 Thread via GitHub
THausherr merged PR #1718: URL: https://github.com/apache/tika/pull/1718 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Bump aws.version from 1.12.698 to 1.12.699 [tika]

2024-04-10 Thread via GitHub
THausherr merged PR #1721: URL: https://github.com/apache/tika/pull/1721 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[PR] Bump aws.version from 1.12.698 to 1.12.699 [tika]

2024-04-10 Thread via GitHub
dependabot[bot] opened a new pull request, #1721: URL: https://github.com/apache/tika/pull/1721 Bumps `aws.version` from 1.12.698 to 1.12.699. Updates `com.amazonaws:aws-java-sdk-s3` from 1.12.698 to 1.12.699 Changelog Sourced from

Re: Bump dependabot to weekly?

2024-04-10 Thread Tim Allison
>you start deleting them reflexively out of your email! Not Tilman!!! Let's move to weekly and see how that works? On Wed, Apr 10, 2024 at 3:57 PM Eric Pugh wrote: > > Hence why I like the monthly unless it’s a special case…. The flood of > updates just means you start deleting them

Re: Bump dependabot to weekly?

2024-04-10 Thread Eric Pugh
Hence why I like the monthly unless it’s a special case…. The flood of updates just means you start deleting them reflexively out of your email! Now, if you have a dependency and you’re maybe actively working on it, and it’s changing quickly, then that might be an argument for daily. > On

AW: Bump dependabot to weekly?

2024-04-10 Thread Tilman Hausherr
I'm fine with daily because this way we can learn ASAP if there are troubles with new dependency versions, although I'm now too busy. Tilman -- Original-Nachricht -- Von: Tim Allison Betreff: Bump dependabot to weekly? Datum: 10.04.2024, 18:08 Uhr An: All, Tilman has been doing heroic

Re: Bump dependabot to weekly?

2024-04-10 Thread Nicholas DiPiazza
Less frequent is good On Wed, Apr 10, 2024, 11:29 AM Eric Pugh wrote: > Or even monthly? Some projects release so frequently that you get many > upgrades between release cycles, so it feels more treadmill-ish…. > > On the Quepid project I changed it to run on the first day of the month, > and

Re: Bump dependabot to weekly?

2024-04-10 Thread Eric Pugh
Or even monthly? Some projects release so frequently that you get many upgrades between release cycles, so it feels more treadmill-ish…. On the Quepid project I changed it to run on the first day of the month, and that’s been plenty ;-). > On Apr 10, 2024, at 12:08 PM, Tim Allison

Bump dependabot to weekly?

2024-04-10 Thread Tim Allison
All, Tilman has been doing heroic work keeping us up to date with dependabot's PRs. Given our pace of releases, would it make sense to backoff to weekly updates? Before running regression tests, we'd run the update plugin to make sure that we're up to date. What do you think? Best,

Re: Checkstyle - ignore line length or just use a bigger value

2024-04-10 Thread Tim Allison
There will be surprises. I'm not sure if the new streaming format will work with checkstyle, etc... We can iterate (or revert if anyone has a problem!) On Wed, Apr 10, 2024 at 11:31 AM Nicholas DiPiazza wrote: > > Thank you so much for making these changes so fast. They will make my > quality

Re: Checkstyle - ignore line length or just use a bigger value

2024-04-10 Thread Nicholas DiPiazza
Thank you so much for making these changes so fast. They will make my quality of life much better On Wed, Apr 10, 2024, 10:03 AM Tim Allison wrote: > I bumped line length to 180 from 120. Let's see if that's enough. > > I'm not sure what the best option is for chained method calls? > "Chained

Re: Checkstyle - ignore line length or just use a bigger value

2024-04-10 Thread Tim Allison
I bumped line length to 180 from 120. Let's see if that's enough. I'm not sure what the best option is for chained method calls? "Chained method calls" -> wrap always? "Wrap first call" -> true "Align with multiline" -> true I just pushed a slightly updated intellij-code-style.xml Let me know

Re: Checkstyle - ignore line length or just use a bigger value

2024-04-10 Thread Nicholas DiPiazza
We could probably double it. and if there's some way to allow the Intellij formatter new line settings to allow multi-line streaming expressions builder() .name("nick") .someOtherStuff("doIt") .build() right now the formatter turns that into 1 line On Wed, Apr 10, 2024 at 5:06 AM Tim

Re: Checkstyle - ignore line length or just use a bigger value

2024-04-10 Thread Tim Allison
Sounds good. What length? On Wed, Apr 10, 2024 at 1:18 AM Nicholas DiPiazza wrote: > > can we bump up the line break to a more reasonable number? > some of the stream expressions start to wrap and wrap and warp forcing me > to use smaller variable names or break down into methods when i'd

Re: [PR] Bump aws.version from 1.12.697 to 1.12.698 [tika]

2024-04-10 Thread via GitHub
THausherr merged PR #1719: URL: https://github.com/apache/tika/pull/1719 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: