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.

Sign up for GitHub

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

Jump to bottom

Pod Targets should be ordered as case insensitive #3684

Closed
orta opened this issue Jun 14, 2015 · 4 comments
Closed

Pod Targets should be ordered as case insensitive #3684

orta opened this issue Jun 14, 2015 · 4 comments
Labels
d1:easy An easy ticket that is a good start for first-time contributors s2:confirmed Issues that have been confirmed by a CocoaPods contributor t1:enhancement Enhancements that have not been picked up yet. Please comment if you plan to work on it

Comments

@orta
Copy link
Member

orta commented Jun 14, 2015

screen shot 2015-06-14 at 16 51 13
@orta orta added t1:enhancement Enhancements that have not been picked up yet. Please comment if you plan to work on it d1:easy An easy ticket that is a good start for first-time contributors s2:confirmed Issues that have been confirmed by a CocoaPods contributor labels Jun 14, 2015
@marcelofabri
Copy link
Contributor

Shouldn't this also apply to other lists? Such as the result of pod install and the Pods group inside Pods.xcodeproj.

https://github.com/CocoaPods/CocoaPods/search?p=1&q=sort&utf8=✓

@segiddins
Copy link
Member

Yup, @marcelofabri ! Basically anywhere we sort based on a string, we should sort based on a normalized version of that string that makes case uniform, and probably strips out unicode characters for their ascii equivalents

emkosz added a commit to emkosz/Xcodeproj that referenced this issue Jul 31, 2015
@emkosz
Sorts the pods in xcode case insensitively
b3b0422 
CocoaPods/CocoaPods#3684
emkosz added a commit to emkosz/Xcodeproj that referenced this issue Jul 31, 2015
@emkosz
Sorts the pods in xcode case insensitively
606ffb4 
CocoaPods/CocoaPods#3684
emkosz added a commit to emkosz/Xcodeproj that referenced this issue Jul 31, 2015
@emkosz
Sort the pods in the project navigator case insensitively
9d0139d 
CocoaPods/CocoaPods#3684
@emkosz
Copy link
Contributor

emkosz commented Jul 31, 2015

Working on this one and should have a PR ready by next week.

@emkosz emkosz mentioned this issue Aug 6, 2015
Merged
emkosz added a commit to emkosz/Xcodeproj that referenced this issue Aug 7, 2015
@emkosz
Sorts the pods in xcode case insensitively
2d1d5a1 
CocoaPods/CocoaPods#3684
emkosz added a commit to emkosz/Xcodeproj that referenced this issue Aug 7, 2015
@emkosz
Sort the pods in the project navigator case insensitively
8eb18e8 
CocoaPods/CocoaPods#3684
emkosz added a commit to emkosz/Xcodeproj that referenced this issue Aug 10, 2015
@emkosz
Sorts the pods in xcode case insensitively
7de4bb3 
CocoaPods/CocoaPods#3684
emkosz added a commit to emkosz/Xcodeproj that referenced this issue Aug 10, 2015
@emkosz
Sort the pods in the project navigator case insensitively
9924a51 
CocoaPods/CocoaPods#3684
@segiddins
Copy link
Member

Done by @emkosz !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
d1:easy An easy ticket that is a good start for first-time contributors s2:confirmed Issues that have been confirmed by a CocoaPods contributor t1:enhancement Enhancements that have not been picked up yet. Please comment if you plan to work on it
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@orta @marcelofabri @segiddins @emkosz