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.

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

Feature Request: Add option to pod install to fail if any warnings are emitted #8155

Open
72A12F4E opened this issue Oct 3, 2018 · 2 comments · May be fixed by #8943
Open

Feature Request: Add option to pod install to fail if any warnings are emitted #8155

72A12F4E opened this issue Oct 3, 2018 · 2 comments · May be fixed by #8943
Labels
d1:easy An easy ticket that is a good start for first-time contributors help wanted Help from new or existing contributors would be greatly appreciated! s7:workaround available A workaround for the issue is available t1:enhancement Enhancements that have not been picked up yet. Please comment if you plan to work on it

Comments

@72A12F4E
Copy link

72A12F4E commented Oct 3, 2018

When doing a pod install, I often get warnings like the following

Pod installation complete! There are 33 dependencies from the Podfile and 63 total pods installed.

[!] `<PBXResourcesBuildPhase UUID=`2E14AAE21E7C267E00C2D465`>` attempted to initialize an object with an unknown UUID. `1A2F35A41F28EE0100BC1A6D` for attribute: `files`. This can be the result of a merge and  the unknown UUID is being discarded.

[!] `<PBXResourcesBuildPhase UUID=`2E14AAE21E7C267E00C2D465`>` attempted to initialize an object with an unknown UUID. `2F062F561F54828700588292` for attribute: `files`. This can be the result of a merge and  the unknown UUID is being discarded.

This is often the result of a bad merge during development. It would be nice if I could make pod install fail if warnings like this come up, so my dev teams can correct them immediately.

@dnkoutso dnkoutso 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 help wanted Help from new or existing contributors would be greatly appreciated! labels Oct 3, 2018
@dnkoutso
Copy link
Contributor

dnkoutso commented Oct 3, 2018

Sure I think its a fairly easy enhancement and starter task for anyone who wants to contribute!

@72A12F4E
Copy link
Author

72A12F4E commented Oct 4, 2018

If anyone else wants a quick workaround for their CI purposes, try this

pod install 2>&1 | tee pod-install-output.txt
if (grep -q '\[!\]' pod-install-output.txt); then
    echo "'pod install' finished with warnings! 😢"
    exit 1
fi

@dnkoutso dnkoutso added the s7:workaround available A workaround for the issue is available label Mar 21, 2019
@ezraberch ezraberch linked a pull request Jun 27, 2019 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
d1:easy An easy ticket that is a good start for first-time contributors help wanted Help from new or existing contributors would be greatly appreciated! s7:workaround available A workaround for the issue is available t1:enhancement Enhancements that have not been picked up yet. Please comment if you plan to work on it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants