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

Fix warnings #118

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ecaselles
Copy link

@ecaselles ecaselles commented Feb 27, 2017

Current version of Tweaks displays warnings in your project when the pod is added. So we have got ahead and fix the warnings in the header files.

Also, since we were here, we thought it would be nice to fix the rest of the warnings in FBTweak and FBTweakExample projects. But happy to roll back these changes (see commits) if not a good idea.

Devs: @emmaviolet & @ecaselles

@facebook-github-bot
Copy link
Contributor

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact cla@fb.com if you have any questions.

@facebook-github-bot
Copy link
Contributor

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@ecaselles
Copy link
Author

ecaselles commented Mar 22, 2017

Not sure why TravisCI build failed. I can't rebuild it either. Any help? 🙏

@ecaselles
Copy link
Author

@grp Any help getting this one merged? 🙏

@wjmelements
Copy link

@grp ptal

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants