Skip to content

Handle .tool-versions' line break on Windows#357

Merged
paulo-ferraz-oliveira merged 2 commits intomainfrom
fix/_tool-versions-split-for-win
Jul 3, 2025
Merged

Handle .tool-versions' line break on Windows#357
paulo-ferraz-oliveira merged 2 commits intomainfrom
fix/_tool-versions-split-for-win

Conversation

@paulo-ferraz-oliveira
Copy link
Collaborator

Description

This'll hopefully fix the bug reported in the linked issue.

Closes #355.

@paulo-ferraz-oliveira paulo-ferraz-oliveira force-pushed the fix/_tool-versions-split-for-win branch 4 times, most recently from b0f166e to 5d50771 Compare July 2, 2025 21:02
@paulo-ferraz-oliveira paulo-ferraz-oliveira force-pushed the fix/_tool-versions-split-for-win branch from 5d50771 to 91c62ff Compare July 2, 2025 21:04
@paulo-ferraz-oliveira
Copy link
Collaborator Author

In the first commit (adds a specific line-ending test) this is failing

+ '27.3.4.1\r'
- '27.3.4.1'

@paulo-ferraz-oliveira paulo-ferraz-oliveira force-pushed the fix/_tool-versions-split-for-win branch from 6932308 to 25df23a Compare July 2, 2025 21:11
@paulo-ferraz-oliveira
Copy link
Collaborator Author

@Hermanverschooten, I think this fixes your issue

Give it a spin, and lemme know so I can cut a release:

uses: erlef/setup-beam@fix/_tool-versions-split-for-win

Cheers.

@paulo-ferraz-oliveira paulo-ferraz-oliveira marked this pull request as ready for review July 2, 2025 21:16
@Hermanverschooten
Copy link

Works like a charm ❤️

@paulo-ferraz-oliveira
Copy link
Collaborator Author

Requesting review for visibility, but this is a simple fix with low impact, so I'm merging to release a version.

@paulo-ferraz-oliveira paulo-ferraz-oliveira merged commit 1fe9179 into main Jul 3, 2025
78 checks passed
@paulo-ferraz-oliveira paulo-ferraz-oliveira deleted the fix/_tool-versions-split-for-win branch July 3, 2025 16:35
@paulo-ferraz-oliveira
Copy link
Collaborator Author

This is released in v1.20.3.

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.

OTP not found when using windows and .tool-versions file

2 participants