Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

[config] fix: add files to 'copy bundle resources', not 'compile sour… #2936

Merged
merged 1 commit into from
Nov 23, 2020

Conversation

cruzach
Copy link
Contributor

@cruzach cruzach commented Nov 23, 2020

…ces' build phase

why

the two instances where this function is used are:

  • adding GoogleService-Info.plist to the project
  • adding localization strings to the project

In both of those instances, when you follow the relevant guides for them, the respective files are added to the "Copy Bundle Resources" build phase, not the "Compile Sources" build phase (google services guide and localization guide)

Placing them in the compile sources build phase causes errors like the one failing this build- https://expo.io/accounts/charliecruzan/builds/v2/9595f8bb-cd42-4208-8c30-c8e3b6f7d05d/logs

test plan

ran eject locally, here's the result:

image

@cruzach cruzach requested a review from brentvatne November 23, 2020 17:25
@brentvatne brentvatne merged commit 76c2ea3 into master Nov 23, 2020
@brentvatne brentvatne deleted the @cruzach/eject-ios-resources branch November 23, 2020 20:14
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.

2 participants