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

Add "Xcode" directory #270

Closed
wants to merge 1 commit into from
Closed

Add "Xcode" directory #270

wants to merge 1 commit into from

Conversation

jbrudvik
Copy link
Contributor

@jbrudvik jbrudvik commented Sep 8, 2014

Create an "Xcode" directory and copy the current contents of the
"Xcode 4" directory to it.

Previously, the "Xcode 4" directory was the only Xcode-related
directory in the repo, which is misleading because the themes
contained in it also work fine in Xcode 6 (beta 5).

Other projects (e.g., Alcatraz) use URLs that link directly to the
existing themes in the "Xcode 4" directory, so instead of removing the
"Xcode 4" directory and breaking everything that might be linking to
those themes, I propose we leave it for now.

Moving forward, instead of supporting each version of Xcode by creating
an separate "Xcode " directory -- which wasn't done for
Xcode 5 or Xcode 6 -- I propose there be a single "Xcode" directory
that supports the latest version of Xcode.

Create an "Xcode" directory and copy the current contents of the
"Xcode 4" directory to it.

Previously, the "Xcode 4" directory was the only Xcode-related
directory in the repo, which is misleading because the themes
contained in it also work fine in Xcode 6 (beta 5).

Other projects (e.g., Alcatraz) use URLs that link directly to the
existing themes in the "Xcode 4" directory, so instead of removing the
"Xcode 4" directory and breaking everything that might be linking to
those themes, I propose we leave it for now.

Moving forward, instead of supporting each version of Xcode by creating
an separate "Xcode <version>" directory -- which wasn't done for
Xcode 5 or Xcode 6 -- I propose there be a single "Xcode" directory
that supports the latest version of Xcode.
@nyaray
Copy link

nyaray commented Sep 14, 2016

Looks like you forgot to add the deletion (that is, renaming) of the old directory.

@jbrudvik
Copy link
Contributor Author

Based on my initial comments, it looks like I intentionally chose to not delete the old directory (Xcode 4):

Other projects (e.g., Alcatraz) use URLs that link directly to the
existing themes in the "Xcode 4" directory, so instead of removing the
"Xcode 4" directory and breaking everything that might be linking to
those themes, I propose we leave it for now.

@nyaray
Copy link

nyaray commented Sep 14, 2016

Oh, misread that. Sorry!

@jbrudvik jbrudvik closed this Mar 18, 2020
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