Skip to content

fix: update node-xcode library #4406

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

Merged
merged 1 commit into from
Feb 28, 2019
Merged

Conversation

KristianDD
Copy link
Contributor

@KristianDD KristianDD commented Feb 27, 2019

PR Checklist

What is the current behavior?

What is the new behavior?

filesRelativeToProject is needed to maintain the default behaviour of the library and keep the library tests green.

The ".notadd" file should be added to the PbxFileReference section, but not to the PBXSourcesBuildPhase group.

Related to #4391

@ghost ghost assigned KristianDD Feb 27, 2019
@ghost ghost added the new PR label Feb 27, 2019
@cla-bot cla-bot bot removed the new PR label Feb 28, 2019
@NativeScript NativeScript deleted a comment from cla-bot bot Feb 28, 2019
@NativeScript NativeScript deleted a comment from cla-bot bot Feb 28, 2019
@NativeScript NativeScript deleted a comment from cla-bot bot Feb 28, 2019
@KristianDD KristianDD force-pushed the kddimitrov/rebase-node-xcode branch from d3cd99b to 4ab889d Compare February 28, 2019 12:25
@ghost ghost added the new PR label Feb 28, 2019
@KristianDD KristianDD force-pushed the kddimitrov/rebase-node-xcode branch from 4ab889d to b2ecf9e Compare February 28, 2019 12:33
@cla-bot cla-bot bot added the cla: yes label Feb 28, 2019
@NativeScript NativeScript deleted a comment from cla-bot bot Feb 28, 2019
@NativeScript NativeScript deleted a comment from cla-bot bot Feb 28, 2019
@KristianDD KristianDD force-pushed the kddimitrov/rebase-node-xcode branch from b2ecf9e to dfe2fc3 Compare February 28, 2019 15:20
@KristianDD
Copy link
Contributor Author

run ci

@KristianDD KristianDD merged commit 32bec2f into master Feb 28, 2019
@KristianDD KristianDD deleted the kddimitrov/rebase-node-xcode branch February 28, 2019 16:20
@ghost ghost removed the new PR label Feb 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants