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

MWRAPPER-139: Remove trailing whitespace from fields in properties #139

Merged
merged 1 commit into from
May 21, 2024

Conversation

ascopes
Copy link
Contributor

@ascopes ascopes commented May 12, 2024

MWRAPPER-139: Remove trailing whitespace from fields in maven-wrapper.properties

This prevents issues due to confusion between CRLF and LF when running in more exotic locations such as MINGW on Windows but using ./mvnw rather than ./mvnw.cmd. A prime example of where this is an issue would be on windows-latest GitHub Actions runners.

Fixes MWRAPPER-139, MWRAPPER-137 (which I didn't spot for some reason when reviewing the
existing tickets on JIRA... might have been a blip in JIRA that didn't display it?).

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

….properties

This prevents issues due to confusion between CRLF and LF when
running in more exotic locations such as MINGW on Windows but
using ./mvnw rather than ./mvnw.cmd. A prime example of where
this is an issue would be on windows-latest GitHub Actions
runners.

Fixes MWRAPPER-139, MWRAPPER-137.
@cstamas cstamas merged commit 1149c75 into apache:master May 21, 2024
20 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.

2 participants