Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
cstamas commented on PR #131: URL: https://github.com/apache/maven-wrapper/pull/131#issuecomment-2061137170 Next attempt: https://github.com/apache/maven-wrapper/pull/132 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
cstamas commented on PR #131: URL: https://github.com/apache/maven-wrapper/pull/131#issuecomment-2061129221 Yup, that is my problem as well: "not sure" :smile: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
jorsol commented on PR #131: URL: https://github.com/apache/maven-wrapper/pull/131#issuecomment-2061121345 Since the default is now `only-mvnw`, maybe the current workaround is to simply remove the lines from:

Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
cstamas commented on PR #131: URL: https://github.com/apache/maven-wrapper/pull/131#issuecomment-2061120774 Oh, got it now. Ok, am closing this one out then, as I missed the topic. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
cstamas closed pull request #131: [MWRAPPER-103] Workaround: demote check to Warning URL: https://github.com/apache/maven-wrapper/pull/131 -- 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

Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
jorsol commented on PR #131: URL: https://github.com/apache/maven-wrapper/pull/131#issuecomment-2061113392 The issue affects the `wrapperSha256Sum` from `mvnw`, not `only-mvnw`, actually the only-mvnw does not validate the `wrapperSha256Sum` since it doesn't download the jar. In the

Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
cstamas commented on PR #131: URL: https://github.com/apache/maven-wrapper/pull/131#issuecomment-2061082617 @jorsol eyes 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

Re: [PR] [MWRAPPER-103] Workaround: demote check to Warning [maven-wrapper]

2024-04-17 Thread via GitHub
cstamas commented on PR #131: URL: https://github.com/apache/maven-wrapper/pull/131#issuecomment-2061082217 @gsmet eyes please. Disclaimer: I am not Win guy, so if you can bring on someone to verify, please do so. -- This is an automated message from the Apache Git Service. To respond to