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

Improve error message when using xcodes select without Xcode installed #351

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ngimelliUW
Copy link

@ngimelliUW ngimelliUW commented Mar 4, 2024

Summary This PR improves the error message given when running xcodes select without any Xcode version installed.

Previously, the error message was: "Not a valid number. Expecting a whole number between 1-0", but ____ was given. Now, if no Xcode is installed, it will say No Xcode version installed. Please run 'xcodes install' and try again

Screenshots

Before After

@ngimelliUW ngimelliUW force-pushed the feature/350/xcodes-select-error-message-improvement-without-xcode-installed branch from 99152cf to 922007b Compare March 4, 2024 22:14
@ngimelliUW ngimelliUW marked this pull request as ready for review March 4, 2024 22:14
@ngimelliUW ngimelliUW changed the title Improve error message when using xcodes select without Xcode install Improve error message when using xcodes select without Xcode installed Mar 4, 2024
Sources/XcodesKit/XcodeSelect.swift Outdated Show resolved Hide resolved
@ngimelliUW ngimelliUW force-pushed the feature/350/xcodes-select-error-message-improvement-without-xcode-installed branch from 922007b to 93c3e77 Compare March 4, 2024 23:12
@ngimelliUW ngimelliUW force-pushed the feature/350/xcodes-select-error-message-improvement-without-xcode-installed branch from 93c3e77 to 9b10bb0 Compare March 4, 2024 23:12
@ngimelliUW
Copy link
Author

@MattKiazyk 👀

@ngimelliUW
Copy link
Author

@MattKiazyk can you please review this?

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