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

Fixes xcode section showing everywhere #1268

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TheNightmanCodeth
Copy link

Description

As per issue #1258, this PR ensures the Xcode prompt is only shown when inside of an Xcode project

@TheNightmanCodeth
Copy link
Author

I've fixed the issue with this section showing in all directories and added functionality so the section works without xcenv installed. The SPACESHIP_XCODE_SHOW_LOCAL/GLOBAL flags do the same thing they did before when xcenv is installed and they do nothing if it isn't.

@denysdovhan denysdovhan added the improvement A PR that make small changes for improving UX, performance, readability, etc label Nov 2, 2022
@@ -9,8 +9,8 @@
# ------------------------------------------------------------------------------

SPACESHIP_XCODE_ASYNC="${SPACESHIP_XCODE_ASYNC=true}"
SPACESHIP_XCODE_SHOW_LOCAL="${SPACESHIP_XCODE_SHOW_LOCAL=true}"
SPACESHIP_XCODE_SHOW_GLOBAL="${SPACESHIP_XCODE_SHOW_GLOBAL=false}"
SPACESHIP_XCODE_SHOW_LOCAL="${SPACESHIP_XCODE_SHOW_LOCAL=false}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't flipping the values of these flags a breaking change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement A PR that make small changes for improving UX, performance, readability, etc
Development

Successfully merging this pull request may close these issues.

None yet

2 participants