Skip to content
This repository has been archived by the owner on Nov 3, 2020. It is now read-only.

Pod name completion while editing Podfile #197

Merged
merged 6 commits into from
Jan 16, 2016
Merged

Pod name completion while editing Podfile #197

merged 6 commits into from
Jan 16, 2016

Conversation

nwest
Copy link
Member

@nwest nwest commented Jan 15, 2016

Implements #19. Autocomplete can be toggled with ESC.
screen shot 2016-01-15 at 4 06 05 pm

This likely broke autocompletion of the Podfile methods (use_frameworks! for e.g). I'll find a way to pull those in to this delegation pattern. https://github.com/nwest/CocoaPods-app/commit/50f69427ee90c9d319fb3b6aaecce8b32e5b7821

@@ -44,6 +54,13 @@ class CPPodfileEditorViewController: NSViewController, NSTextViewDelegate {
syntaxChecker.textDidChange(NSNotification(name: "", object: nil))
}

func completions() -> [AnyObject]! {
Copy link
Member

Choose a reason for hiding this comment

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

[String]

Copy link
Member Author

Choose a reason for hiding this comment

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

Unfortunately, this is how it's declared in SMLAutoCompleteDelegate. If I change the return type in the implementation, the build errors for non protocol conformance.

Copy link
Member

Choose a reason for hiding this comment

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

ah boo :(

Copy link
Member

Choose a reason for hiding this comment

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

how often is this called? Is it left to Fragaria to deal with scoping?

Copy link
Member

Choose a reason for hiding this comment

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

ah yeah, totally looks like it.

@orta
Copy link
Member

orta commented Jan 15, 2016

Ah yeah!

@orta
Copy link
Member

orta commented Jan 15, 2016

This likely broke autocompletion of the Podfile methods (use_frameworks! for e.g). I'll find a way to pull those in to this delegation pattern.

I think you need to pull them out of here: https://github.com/CocoaPods/CocoaPods-app/blob/master/app/CocoaPods/Supporting%20Files/Syntax%20Definitions/Podfile.plist

and add them to the array you're showing.

@orta
Copy link
Member

orta commented Jan 16, 2016

Great, so this will will need a rebase and it's good to go 👍

@segiddins
Copy link
Member

@nwest please rebase instead of merging master to keep history clean?

orta added a commit that referenced this pull request Jan 16, 2016
Pod name completion while editing Podfile
@orta orta merged commit 0327582 into CocoaPods:master Jan 16, 2016
@orta
Copy link
Member

orta commented Jan 16, 2016

🎉

@nwest
Copy link
Member Author

nwest commented Jan 16, 2016

@segiddins ah, I'll know to rebase in the future.

@nwest nwest deleted the pod-list-bridging branch January 16, 2016 14:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants