[PR] Refactor code smells [commons-collections]

2024-03-29 Thread via GitHub


Keval-Gandevia opened a new pull request, #471:
URL: https://github.com/apache/commons-collections/pull/471

   **Implementation smell resolved.**
   
   | Implementation smell | Class name | Method name |
   | --- | --- | --- |
   |Extract method | CollectionUtils | get |
   | Decompose conditional | ListUtils | isEqualList |
   | Rename method / variable | SetUniqueList | set |
   
   


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Test cases for iterators package, map package, and collection4 package [commons-collections]

2024-03-29 Thread via GitHub


Keval-Gandevia closed pull request #460: Test cases for iterators package, map 
package, and collection4 package
URL: https://github.com/apache/commons-collections/pull/460


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] refactor: required as per Assignment 2 - Task 2 - Set 1 [commons-jxpath]

2024-03-29 Thread via GitHub


garydgregory commented on PR #131:
URL: https://github.com/apache/commons-jxpath/pull/131#issuecomment-2027898550

   -1 I don't plan on reviewing school assignments. 
   
   If you are looking to help this project you can look at the bug tracker 
here: https://issues.apache.org/jira/projects/JXPATH


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] refactor: required as per Assignment 2 - Task 2 - Set 1 [commons-jxpath]

2024-03-29 Thread via GitHub


JinxLordMAW opened a new pull request, #131:
URL: https://github.com/apache/commons-jxpath/pull/131

   (no comment)


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] refactor: code smell assignment 2 - Part 2A [commons-jxpath]

2024-03-29 Thread via GitHub


JinxLordMAW closed pull request #130: refactor: code smell assignment 2 - Part 
2A
URL: https://github.com/apache/commons-jxpath/pull/130


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-beanutils]

2024-03-29 Thread via GitHub


garydgregory merged PR #228:
URL: https://github.com/apache/commons-beanutils/pull/228


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-validator]

2024-03-29 Thread via GitHub


garydgregory merged PR #200:
URL: https://github.com/apache/commons-validator/pull/200


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-email]

2024-03-29 Thread via GitHub


garydgregory merged PR #232:
URL: https://github.com/apache/commons-email/pull/232


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons-io:commons-io from 2.15.1 to 2.16.0 [commons-text]

2024-03-29 Thread via GitHub


garydgregory merged PR #522:
URL: https://github.com/apache/commons-text/pull/522


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons.bytebuddy.version from 1.14.12 to 1.14.13 [commons-text]

2024-03-29 Thread via GitHub


garydgregory merged PR #521:
URL: https://github.com/apache/commons-text/pull/521


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-text]

2024-03-29 Thread via GitHub


garydgregory merged PR #520:
URL: https://github.com/apache/commons-text/pull/520


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-text]

2024-03-29 Thread via GitHub


garydgregory commented on PR #520:
URL: https://github.com/apache/commons-text/pull/520#issuecomment-2027748857

   @dependabot rebase 


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons.bytebuddy.version from 1.14.12 to 1.14.13 [commons-text]

2024-03-29 Thread via GitHub


garydgregory commented on PR #521:
URL: https://github.com/apache/commons-text/pull/521#issuecomment-2027748701

   @dependabot rebase 


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons-io:commons-io from 2.15.1 to 2.16.0 [commons-text]

2024-03-29 Thread via GitHub


garydgregory commented on PR #522:
URL: https://github.com/apache/commons-text/pull/522#issuecomment-2027748156

   @dependabot rebase 


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-bcel]

2024-03-29 Thread via GitHub


garydgregory merged PR #296:
URL: https://github.com/apache/commons-bcel/pull/296


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-rdf]

2024-03-29 Thread via GitHub


garydgregory merged PR #211:
URL: https://github.com/apache/commons-rdf/pull/211


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-fileupload]

2024-03-29 Thread via GitHub


garydgregory merged PR #293:
URL: https://github.com/apache/commons-fileupload/pull/293


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons-logging:commons-logging from 1.3.0 to 1.3.1 [commons-jxpath]

2024-03-29 Thread via GitHub


garydgregory merged PR #129:
URL: https://github.com/apache/commons-jxpath/pull/129


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-jxpath]

2024-03-29 Thread via GitHub


garydgregory merged PR #127:
URL: https://github.com/apache/commons-jxpath/pull/127


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Support for Java 16 Record feature [commons-bcel]

2024-03-29 Thread via GitHub


paulk-asert commented on code in PR #290:
URL: https://github.com/apache/commons-bcel/pull/290#discussion_r1544870277


##
src/test/java/org/apache/bcel/classfile/RecordTestCase.java:
##
@@ -0,0 +1,120 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+package org.apache.bcel.classfile;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import java.io.File;
+import java.io.IOException;
+
+import org.apache.bcel.AbstractTestCase;
+import org.apache.bcel.util.SyntheticRepository;
+import org.apache.bcel.visitors.CountingVisitor;
+import org.junit.jupiter.api.Test;
+
+public class RecordTestCase extends AbstractTestCase {
+
+/**
+ * A record type, once compiled, should result in a class file that is
+ * marked such that we can determine from the access flags
+ * (through BCEL) that it is in fact a record.
+ *
+ * @throws IOException
+ * @throws ClassFormatException
+ */
+@Test
+public void testRecordClassSaysItIs() throws ClassNotFoundException, 
ClassFormatException, IOException {
+final JavaClass clazz = new 
ClassParser("src/test/resources/record/SimpleRecord.class").parse();
+assertTrue(clazz.isRecord(), "Expected SimpleEnum class to say it was 
a record - but it didn't !");

Review Comment:
   SimpleEnum -> SimpleRecord in test failure message



-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [IMAGING-319] Fix for lost first character on String [commons-imaging]

2024-03-29 Thread via GitHub


kinow commented on PR #359:
URL: https://github.com/apache/commons-imaging/pull/359#issuecomment-2027695615

   > @kinow Great news, thanks!
   > 
   > Maybe we can use the image attached to 
https://issues.apache.org/jira/browse/IMAGING-319. I just asked the reporter.
   
   Excellent idea! That'd save us a lot of time, thanks for the initiative, 
@StefanOltmann :+1: 


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (IMAGING-319) updateExifMetadataLossless lost the first character of a String

2024-03-29 Thread Stefan Oltmann (Jira)


[ 
https://issues.apache.org/jira/browse/IMAGING-319?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17832282#comment-17832282
 ] 

Stefan Oltmann commented on IMAGING-319:


Hi [~yangsicheng],

we have a fix for the problem, but we haven't it commited to the sources as we 
lack a test image for the unit tests.

Is the image you attached to this ticket yours? Do you own the rights to it? If 
so, would you allow us to add it to the projects sources (for unit tests) under 
Apache 2 license?

@[~kinow] FYI

> updateExifMetadataLossless lost the first character of a String
> ---
>
> Key: IMAGING-319
> URL: https://issues.apache.org/jira/browse/IMAGING-319
> Project: Commons Imaging
>  Issue Type: Bug
>  Components: Format: JPEG
>Affects Versions: 1.0-alpha2
>Reporter: Sicheng Yang
>Assignee: Bruno P. Kinoshita
>Priority: Major
> Attachments: Screen Shot 2021-11-26 at 4.01.06 PM-1.png, Screen Shot 
> 2021-11-26 at 4.01.21 PM-1.png, iPhone12-geotag.JPG
>
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> I try to use TiffOutputSet to generate a new image. However, if a tag that 
> contains String, the program may miss the first character of the String.
>  
> import java.io.*;
> import org.apache.commons.imaging.ImageReadException;
> import org.apache.commons.imaging.ImageWriteException;
> import org.apache.commons.imaging.Imaging;
> import org.apache.commons.imaging.common.ImageMetadata;
> import org.apache.commons.imaging.formats.jpeg.JpegImageMetadata;
> import org.apache.commons.imaging.formats.jpeg.exif.ExifRewriter;
> import org.apache.commons.imaging.formats.tiff.TiffImageMetadata;
> import org.apache.commons.imaging.formats.tiff.write.TiffOutputSet;
> public class LibraryTest {
>     public static void main(String[] args) throws ImageReadException, 
> IOException, ImageWriteException {
>         File source = new File("./assets/iPhone12-geotag.JPG");
>         File result = new 
> File("./assets/results/editted-iPhone12-geotag.JPG");
>         final ImageMetadata metadata = Imaging.getMetadata(source);
>         final JpegImageMetadata jpegMetadata = (JpegImageMetadata) metadata;
>         final TiffImageMetadata exif = jpegMetadata.getExif();
>         TiffOutputSet outputSet = exif.getOutputSet();
>         BufferedOutputStream bufferedOutputStream = new 
> BufferedOutputStream(new FileOutputStream(result));
>         new ExifRewriter().updateExifMetadataLossless(source, 
> bufferedOutputStream, outputSet);
>     }
> }
>  
> This is the sample code.
> Tag value in original image
> !image-2021-11-26-16-01-58-645.png!
> Tag value in output image
> !image-2021-11-26-16-04-12-185.png!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] [IMAGING-319] Fix for lost first character on String [commons-imaging]

2024-03-29 Thread via GitHub


StefanOltmann commented on PR #359:
URL: https://github.com/apache/commons-imaging/pull/359#issuecomment-2027568778

   @kinow Great news, thanks!
   
   The reporter of https://issues.apache.org/jira/browse/IMAGING-319 also 
attached a file to his ticket. It looks like a private iPhone image to me. We 
could ask him if he might contribute it to the project under Apache 2 license.


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-daemon]

2024-03-29 Thread via GitHub


garydgregory merged PR #154:
URL: https://github.com/apache/commons-daemon/pull/154


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-configuration]

2024-03-29 Thread via GitHub


garydgregory merged PR #395:
URL: https://github.com/apache/commons-configuration/pull/395


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-exec]

2024-03-29 Thread via GitHub


garydgregory merged PR #173:
URL: https://github.com/apache/commons-exec/pull/173


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-imaging]

2024-03-29 Thread via GitHub


garydgregory merged PR #381:
URL: https://github.com/apache/commons-imaging/pull/381


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-codec]

2024-03-29 Thread via GitHub


garydgregory merged PR #260:
URL: https://github.com/apache/commons-codec/pull/260


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-lang]

2024-03-29 Thread via GitHub


garydgregory merged PR #1192:
URL: https://github.com/apache/commons-lang/pull/1192


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [CLI-329] Support "Deprecated" CLI Options [commons-cli]

2024-03-29 Thread via GitHub


garydgregory merged PR #252:
URL: https://github.com/apache/commons-cli/pull/252


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [CLI-329] Support "Deprecated" CLI Options [commons-cli]

2024-03-29 Thread via GitHub


garydgregory commented on PR #252:
URL: https://github.com/apache/commons-cli/pull/252#issuecomment-2027439274

   @epugh 
   We're all set then, merged!


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [IMAGING-319] Fix for lost first character on String [commons-imaging]

2024-03-29 Thread via GitHub


kinow commented on PR #359:
URL: https://github.com/apache/commons-imaging/pull/359#issuecomment-2027422945

   Reviewed #366, I think that one can be merged. That pull request fixes a 
`@Disabled` test that does the roud-trip test for EXIF data.
   
   Once that's merged, and the test is re-enabled, there is a test image in 
#366 that should fail. I think here we have to:
   
   1. Wait for #366 to be merged
   2. Use the test image to confirm the test fails for that image
   3. Check out this branch, re-run the test with the new image
   4. Review the license for that image, and if we really cannot use it, then 
decide what to do (I would spend some time checking if a similar image can be 
crafted or found somewhere else)
   5. Once the image is added, confirm the code here is being covered by the 
test+new image, and approve and merge! :rocket:
   
   Cheers


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] BCEL-372: Optimized code AttributeHTML [commons-bcel]

2024-03-29 Thread via GitHub


TYzzt closed pull request #294: BCEL-372: Optimized code AttributeHTML
URL: https://github.com/apache/commons-bcel/pull/294


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [IMAGING-351] ExifRewriterRoundtripTest fixed, completed & enabled [commons-imaging]

2024-03-29 Thread via GitHub


kinow commented on PR #366:
URL: https://github.com/apache/commons-imaging/pull/366#issuecomment-2027412302

   And tests passed after re-running the failed jobs on GitHub Actions 
:man_shrugging: 


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] CLI-329: introduce to Options the tracking of deprecated short and long opts [commons-cli]

2024-03-29 Thread via GitHub


epugh commented on PR #251:
URL: https://github.com/apache/commons-cli/pull/251#issuecomment-2027377978

   Closing in favour of #252


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] CLI-329: introduce to Options the tracking of deprecated short and long opts [commons-cli]

2024-03-29 Thread via GitHub


epugh closed pull request #251: CLI-329: introduce to Options the tracking of 
deprecated short and long opts
URL: https://github.com/apache/commons-cli/pull/251


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [CLI-329] Support "Deprecated" CLI Options [commons-cli]

2024-03-29 Thread via GitHub


epugh commented on PR #252:
URL: https://github.com/apache/commons-cli/pull/252#issuecomment-2027377406

   @garydgregory  I am going to close my PR in favour of this one...   What can 
I do to help move this along?   


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (IO-851) FileSystemUtils no longer throws IllegalStateException

2024-03-29 Thread Sebb (Jira)


[ 
https://issues.apache.org/jira/browse/IO-851?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17832224#comment-17832224
 ] 

Sebb commented on IO-851:
-

That's IllegalArgumentException, not IllegalStateException.

I just did a search for IllegalStateException and it only appears in Javadoc.

> FileSystemUtils no longer throws IllegalStateException
> --
>
> Key: IO-851
> URL: https://issues.apache.org/jira/browse/IO-851
> Project: Commons IO
>  Issue Type: Bug
>Reporter: Sebb
>Priority: Minor
>
> The recent change to use NIO means that the code no longer throws 
> IllegalStateException, contrary to the Javadoc:
> https://github.com/apache/commons-io/commit/59129bcf283ffa5a46d0872e173a4e037575c581



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] [IMAGING-351] ExifRewriterRoundtripTest fixed, completed & enabled [commons-imaging]

2024-03-29 Thread via GitHub


kinow commented on PR #366:
URL: https://github.com/apache/commons-imaging/pull/366#issuecomment-2027349014

   Build errors are on macos (works on macos + java 8, fails for java 11).
   
   >Error:  Java heap space
   
   I will spend some minutes checking if there's any Maven setting that we can 
use to fix the build, but I think that shouldn't be a blocker.
   
   I still want to test this part from @StefanOltmann 's comment before merging
   
   >Drop this 
[testfile.jpg](https://github.com/apache/commons-imaging/assets/9299303/2826b4ee-473a-45fb-b9fc-2168b9ae3fca)
 into src/test/resources/data/images/jpg to make the test fail.
   
   Then I believe a good course of action here would be to merge this to get 
the test passing (thanks Stefan!), and continue working on #359. If I recall 
correctly we may not be able to include the test file, so I will check whether 
I can find another one, find another way to test that, or check with 
legal/others other options (no need to worry about this for now).


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] BCEL-372: Optimized code AttributeHTML [commons-bcel]

2024-03-29 Thread via GitHub


garydgregory commented on PR #294:
URL: https://github.com/apache/commons-bcel/pull/294#issuecomment-2027314695

   This is the lowest priority for me ATM since it is not a bug fix, it's just 
moving stuff around.


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] BCEL-372: Optimized code AttributeHTML [commons-bcel]

2024-03-29 Thread via GitHub


TYzzt commented on PR #294:
URL: https://github.com/apache/commons-bcel/pull/294#issuecomment-2027257996

   Hi @garydgregory ,Can you review the code for me, please?


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons.bytebuddy.version from 1.14.12 to 1.14.13 [commons-io]

2024-03-29 Thread via GitHub


garydgregory merged PR #605:
URL: https://github.com/apache/commons-io/pull/605


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons-io:commons-io from 2.15.1 to 2.16.0 [commons-scxml]

2024-03-29 Thread via GitHub


garydgregory merged PR #210:
URL: https://github.com/apache/commons-scxml/pull/210


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons-io:commons-io from 2.15.1 to 2.16.0 [commons-net]

2024-03-29 Thread via GitHub


garydgregory merged PR #236:
URL: https://github.com/apache/commons-net/pull/236


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump org.apache.logging.log4j:log4j-core from 2.23.0 to 2.23.1 [commons-jcs]

2024-03-29 Thread via GitHub


garydgregory merged PR #229:
URL: https://github.com/apache/commons-jcs/pull/229


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons-io:commons-io from 2.15.1 to 2.16.0 [commons-release-plugin]

2024-03-29 Thread via GitHub


garydgregory merged PR #253:
URL: https://github.com/apache/commons-release-plugin/pull/253


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump org.ow2.asm:asm-util from 9.6 to 9.7 [commons-pool]

2024-03-29 Thread via GitHub


garydgregory merged PR #294:
URL: https://github.com/apache/commons-pool/pull/294


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-csv]

2024-03-29 Thread via GitHub


garydgregory merged PR #409:
URL: https://github.com/apache/commons-csv/pull/409


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-collections]

2024-03-29 Thread via GitHub


garydgregory merged PR #470:
URL: https://github.com/apache/commons-collections/pull/470


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Resolved] (TEXT-175) WordUtils.capitalizeFully(String) only splits by spaces, instead of any whitespace

2024-03-29 Thread Gary D. Gregory (Jira)


 [ 
https://issues.apache.org/jira/browse/TEXT-175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gary D. Gregory resolved TEXT-175.
--
Fix Version/s: 1.11.1
   Resolution: Fixed

> WordUtils.capitalizeFully(String) only splits by spaces, instead of any 
> whitespace
> --
>
> Key: TEXT-175
> URL: https://issues.apache.org/jira/browse/TEXT-175
> Project: Commons Text
>  Issue Type: Bug
>Affects Versions: 1.2, 1.8
>Reporter: David Lavati
>Priority: Minor
> Fix For: 1.11.1
>
>  Time Spent: 1h
>  Remaining Estimate: 0h
>
> {color:#172b4d}Due to the performance improvements in TEXT-98, this no longer 
> works as originally designed:{color}
>  * 
> {color:#172b4d}{{org.apache.commons.text.WordUtils.capitalizeFully("a\tb\nc")}}{color}
>  ** should return "A\t\B\nC", but returns "A\t\b\nc" instead
>  * {color:#172b4d}{{org.apache.commons.text.WordUtils.capitalizeFully("a b 
> c")}}{color}
>  ** still works as "A B C"
>  
> A possible option while keeping the improvements would be to create a static 
> collection of whitespace codepoints and
>  * either include that into the set, instead of just the single whitespace, 
> as done at 
> [https://github.com/apache/commons-text/commit/fb6d5935451397c561bd52cf1d483975f83b2c7b#diff-9222b0893e00b43fd6519d95e395f9b3R892]
>  * or if that touches too many things, call {{capitalize(String,char[])}}  
> inside capitalizeFully with these chars as delimiters by default.
>  
> Tested in 1.8



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] [TEXT-175] Fix regression for determining whitespace in WordUtils [commons-text]

2024-03-29 Thread via GitHub


garydgregory merged PR #519:
URL: https://github.com/apache/commons-text/pull/519


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [TEXT-175] Fix regression for determining whitespace in WordUtils [commons-text]

2024-03-29 Thread via GitHub


garydgregory commented on PR #519:
URL: https://github.com/apache/commons-text/pull/519#issuecomment-2027175627

   k, then this looks good.
   


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-crypto]

2024-03-29 Thread via GitHub


garydgregory merged PR #322:
URL: https://github.com/apache/commons-crypto/pull/322


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump org.moditect:moditect-maven-plugin from 1.2.0.Final to 1.2.1.Final [commons-parent]

2024-03-29 Thread via GitHub


garydgregory merged PR #390:
URL: https://github.com/apache/commons-parent/pull/390


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump org.cyclonedx:cyclonedx-maven-plugin from 2.7.11 to 2.8.0 [commons-parent]

2024-03-29 Thread via GitHub


garydgregory merged PR #389:
URL: https://github.com/apache/commons-parent/pull/389


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-pool]

2024-03-29 Thread via GitHub


garydgregory merged PR #293:
URL: https://github.com/apache/commons-pool/pull/293


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (IO-851) FileSystemUtils no longer throws IllegalStateException

2024-03-29 Thread Gary D. Gregory (Jira)


[ 
https://issues.apache.org/jira/browse/IO-851?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17832183#comment-17832183
 ] 

Gary D. Gregory commented on IO-851:


But it does: 
[https://github.com/apache/commons-io/commit/59129bcf283ffa5a46d0872e173a4e037575c581#diff-db759730c0e3c4c6cf1c3489f71f8ba5f68cd035a1164590e576c1632e3cfb11R175]

 

> FileSystemUtils no longer throws IllegalStateException
> --
>
> Key: IO-851
> URL: https://issues.apache.org/jira/browse/IO-851
> Project: Commons IO
>  Issue Type: Bug
>Reporter: Sebb
>Priority: Minor
>
> The recent change to use NIO means that the code no longer throws 
> IllegalStateException, contrary to the Javadoc:
> https://github.com/apache/commons-io/commit/59129bcf283ffa5a46d0872e173a4e037575c581



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Resolved] (CONFIGURATION-843) Methods should not be empty

2024-03-29 Thread Gary D. Gregory (Jira)


 [ 
https://issues.apache.org/jira/browse/CONFIGURATION-843?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gary D. Gregory resolved CONFIGURATION-843.
---
Fix Version/s: 2.10.2
   Resolution: Fixed

> Methods should not be empty
> ---
>
> Key: CONFIGURATION-843
> URL: https://issues.apache.org/jira/browse/CONFIGURATION-843
> Project: Commons Configuration
>  Issue Type: Task
>Reporter: zhao tao
>Priority: Minor
> Fix For: 2.10.2
>
>
> An empty method is generally considered bad practice and can lead to 
> confusion, readability, and maintenance issues. Empty methods bring no 
> functionality and are misleading to others as they might think the method 
> implementation fulfills a specific and identified requirement.
> There are several reasons for a method not to have a body:



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] CONFIGURATION-843: Methods should not be empty [commons-configuration]

2024-03-29 Thread via GitHub


garydgregory merged PR #393:
URL: https://github.com/apache/commons-configuration/pull/393


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump commons-io:commons-io from 2.15.1 to 2.16.0 [commons-vfs]

2024-03-29 Thread via GitHub


garydgregory merged PR #510:
URL: https://github.com/apache/commons-vfs/pull/510


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [IMAGING-351] ExifRewriterRoundtripTest fixed, completed & enabled [commons-imaging]

2024-03-29 Thread via GitHub


kinow commented on PR #366:
URL: https://github.com/apache/commons-imaging/pull/366#issuecomment-2027097242

   - Replaced test code from `TiffOutputField` by a public getter/setter for 
the bytes data (the getter is a copy, like in other formats)
   - Updated tests to use `assertArrayEquals` from JUnit instead of custom test 
code
   - Updated a few lines to match the formatting of other classes
   - Added link to the issue/PR where the test was added (for traceability)
   - Added Javadocs to public methods changed
   
   I added a second commit so we can still revert/remove things if needed, but 
if there are no objections I think this should be ready to be merged as soon as 
CI is happy (will take a look later, gotta go :wave: :runner: )
   
   p.s. will hold merging this until the current ongoing release is out, I 
think, to avoid causing any issues with the release progress.


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-vfs]

2024-03-29 Thread via GitHub


garydgregory merged PR #509:
URL: https://github.com/apache/commons-vfs/pull/509


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-logging]

2024-03-29 Thread via GitHub


garydgregory merged PR #238:
URL: https://github.com/apache/commons-logging/pull/238


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-logging]

2024-03-29 Thread via GitHub


garydgregory merged PR #237:
URL: https://github.com/apache/commons-logging/pull/237


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-dbutils]

2024-03-29 Thread via GitHub


garydgregory merged PR #254:
URL: https://github.com/apache/commons-dbutils/pull/254


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-release-plugin]

2024-03-29 Thread via GitHub


garydgregory merged PR #255:
URL: https://github.com/apache/commons-release-plugin/pull/255


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-dbutils]

2024-03-29 Thread via GitHub


garydgregory merged PR #255:
URL: https://github.com/apache/commons-dbutils/pull/255


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-release-plugin]

2024-03-29 Thread via GitHub


garydgregory merged PR #256:
URL: https://github.com/apache/commons-release-plugin/pull/256


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-jcs]

2024-03-29 Thread via GitHub


garydgregory merged PR #231:
URL: https://github.com/apache/commons-jcs/pull/231


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-cli]

2024-03-29 Thread via GitHub


garydgregory merged PR #254:
URL: https://github.com/apache/commons-cli/pull/254


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-jcs]

2024-03-29 Thread via GitHub


garydgregory merged PR #230:
URL: https://github.com/apache/commons-jcs/pull/230


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-cli]

2024-03-29 Thread via GitHub


garydgregory merged PR #253:
URL: https://github.com/apache/commons-cli/pull/253


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-build-plugin]

2024-03-29 Thread via GitHub


garydgregory merged PR #238:
URL: https://github.com/apache/commons-build-plugin/pull/238


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-build-plugin]

2024-03-29 Thread via GitHub


garydgregory merged PR #239:
URL: https://github.com/apache/commons-build-plugin/pull/239


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-compress]

2024-03-29 Thread via GitHub


garydgregory merged PR #506:
URL: https://github.com/apache/commons-compress/pull/506


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-compress]

2024-03-29 Thread via GitHub


garydgregory merged PR #507:
URL: https://github.com/apache/commons-compress/pull/507


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-digester]

2024-03-29 Thread via GitHub


garydgregory merged PR #145:
URL: https://github.com/apache/commons-digester/pull/145


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-digester]

2024-03-29 Thread via GitHub


garydgregory merged PR #144:
URL: https://github.com/apache/commons-digester/pull/144


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-dbcp]

2024-03-29 Thread via GitHub


garydgregory merged PR #370:
URL: https://github.com/apache/commons-dbcp/pull/370


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-dbcp]

2024-03-29 Thread via GitHub


garydgregory merged PR #369:
URL: https://github.com/apache/commons-dbcp/pull/369


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump codecov/codecov-action from 4.1.0 to 4.1.1 [commons-net]

2024-03-29 Thread via GitHub


garydgregory merged PR #234:
URL: https://github.com/apache/commons-net/pull/234


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Bump github/codeql-action from 3.24.8 to 3.24.9 [commons-net]

2024-03-29 Thread via GitHub


garydgregory merged PR #235:
URL: https://github.com/apache/commons-net/pull/235


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [TEXT-175] Fix regression for determining whitespace in WordUtils [commons-text]

2024-03-29 Thread via GitHub


kinow commented on PR #519:
URL: https://github.com/apache/commons-text/pull/519#issuecomment-2027018030

   > Do you have any thoughts on this method treating whitespace a certain way 
vs. what the rest of the class does, for example, abbreviate()? IOW, is it OK 
for this PR to only change this method in this manner?
   
   From what you described (haven't checked out the code) I'd agree it'd be 
best to keep things homogeneous.
   
   However, if it is a regression in a recent release, and users from previous 
releases want to have the old behavior, I think it'd be fair to either fix the 
regression, or fix the regression & offer another method with same/different 
behavior. Again, haven't checked out the code (old & new), so saying this based 
on the comment from JIRA:
   
   >Due to the performance improvements in 
[TEXT-98](https://issues.apache.org/jira/browse/TEXT-98), this no longer works 
as originally designed:
   


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (IO-851) FileSystemUtils no longer throws IllegalStateException

2024-03-29 Thread Sebb (Jira)
Sebb created IO-851:
---

 Summary: FileSystemUtils no longer throws IllegalStateException
 Key: IO-851
 URL: https://issues.apache.org/jira/browse/IO-851
 Project: Commons IO
  Issue Type: Bug
Reporter: Sebb


The recent change to use NIO means that the code no longer throws 
IllegalStateException, contrary to the Javadoc:

https://github.com/apache/commons-io/commit/59129bcf283ffa5a46d0872e173a4e037575c581



--
This message was sent by Atlassian Jira
(v8.20.10#820010)