[GitHub] struts pull request #161: WW-4840 Build Fails Due to Unused com.sun Import

2017-08-04 Thread swiftelan
GitHub user swiftelan opened a pull request: https://github.com/apache/struts/pull/161 WW-4840 Build Fails Due to Unused com.sun Import [WW-4840](https://issues.apache.org/jira/browse/WW-4840) Build Fails Due to Unused com.sun Import You can merge this pull request into a Git

[GitHub] struts pull request #160: WW-4839 JakartaStreamMultiPartRequest Should Honor...

2017-08-04 Thread swiftelan
GitHub user swiftelan opened a pull request: https://github.com/apache/struts/pull/160 WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.… [WW-4839](https://issues.apache.org/jira/browse/WW-4839) JakartaStreamMultiPartRequest Should

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-19 Thread swiftelan
Github user swiftelan commented on the pull request: https://github.com/apache/struts/pull/75#issuecomment-172970495 @aleksandr-m An annotation attribute that is defined as an array does not have to have the curly braces to define a single value for that attribute

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-19 Thread swiftelan
Github user swiftelan commented on the pull request: https://github.com/apache/struts/pull/75#issuecomment-172880314 Annotations allow multivalued attributes in the form of arrays. Using this makes it obvious that you can supply multiple result names for a single result configuration

[GitHub] struts pull request: WW-4515

2015-06-17 Thread swiftelan
GitHub user swiftelan opened a pull request: https://github.com/apache/struts/pull/42 WW-4515 Convert try blocks to try-with-resources You can merge this pull request into a Git repository by running: $ git pull https://github.com/swiftelan/struts feature/WW-4515

[GitHub] struts pull request: WW-4503 Build Struts with JDK 7

2015-05-27 Thread swiftelan
Github user swiftelan commented on the pull request: https://github.com/apache/struts/pull/39#issuecomment-105947844 Moving to Java 7 requires the new annotation parser or the removal of the maven-apt-plugin. The annotation parser [job](https://builds.apache.org/view/S-Z/view/Struts

[GitHub] struts pull request: WW-4503 Build Struts with JDK 7

2015-05-26 Thread swiftelan
Github user swiftelan closed the pull request at: https://github.com/apache/struts/pull/39 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts pull request: WW-4503 Build Struts with JDK 7

2015-05-26 Thread swiftelan
Github user swiftelan commented on the pull request: https://github.com/apache/struts/pull/39#issuecomment-105621794 I understand. I merged changes from the GitHub mirror to my fork. I didn't know that it would update the existing pull request, that was not my intention. --- If

[GitHub] struts pull request: WW-4503 Build Struts with JDK 7

2015-05-22 Thread swiftelan
GitHub user swiftelan opened a pull request: https://github.com/apache/struts/pull/39 WW-4503 Build Struts with JDK 7 You can merge this pull request into a Git repository by running: $ git pull https://github.com/swiftelan/struts master Alternatively you can review and