-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix Swift code completion for Development Pods #4106
Conversation
@neonichu ping on this? |
Actually, I'm not sure we can safely do this, as won't this make all the added paths absolute instead of relative? Can you double check? |
Will do. |
Scratch my previous comment, this should work |
@neonichu ping on this? |
acde0c3
to
ca62bcd
Compare
👍 |
ca62bcd
to
67e4aca
Compare
@@ -179,13 +179,13 @@ def pod_support_files_group(pod_name, dir) | |||
# @return [PBXFileReference] The new file reference. | |||
# | |||
def add_file_reference(absolute_path, group, reflect_file_system_structure = false) | |||
file_path_name = Pathname.new(absolute_path) | |||
file_path_name = absolute_path.is_a?(Pathname) ? absolute_path : Pathname.new(absolute_path) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this change necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not really, but it made stubbing things on a specific Pathname
instance possible :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gotcha 👍
…pods Fix Swift code completion for Development Pods
Closes #3777.