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

Fix validator creation path issue on windows #6271

Merged
merged 2 commits into from
Jun 15, 2020
Merged

Conversation

0xKiwi
Copy link
Contributor

@0xKiwi 0xKiwi commented Jun 15, 2020

What type of PR is this?
Bug fix

What does this PR do? Why is it needed?
This PR fixes an issue on windows that when reading from the terminal, the command primpt would inject unsupported line endings, and mess up the path entered, preventing windows users from using the script and bat script properly.

Which issues(s) does this PR fix?

Fixes #5456

@0xKiwi 0xKiwi requested a review from a team as a code owner June 15, 2020 17:24
@0xKiwi 0xKiwi changed the title Fix validator creation path issue Fix validator creation path issue on windows Jun 15, 2020
@0xKiwi 0xKiwi added the Ready For Review A pull request ready for code review label Jun 15, 2020
@prestonvanloon prestonvanloon merged commit bf548d1 into master Jun 15, 2020
@delete-merged-branch delete-merged-branch bot deleted the fix-validator-path branch June 15, 2020 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Validator Path Issue
2 participants