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

Add analyze action to .travis.yml xcodebuild calls #52

Closed
2 tasks done
benasher44 opened this issue Oct 12, 2016 · 4 comments
Closed
2 tasks done

Add analyze action to .travis.yml xcodebuild calls #52

benasher44 opened this issue Oct 12, 2016 · 4 comments

Comments

@benasher44
Copy link
Contributor

benasher44 commented Oct 12, 2016

New issue checklist

Steps

  • Fix remaining analyzer warnings.
  • Add the analyze action to xcodebuild calls in .travis.yml, and ensure that builds that fail the analyzer fail the build.
@benasher44
Copy link
Contributor Author

benasher44 commented Oct 12, 2016

Fixed one analyzer issue in #51

@rnystrom
Copy link
Contributor

Nice, this sounds really helpful. I imagine once the build is working we need to squish all the analyzer warnings before adding it to the .travis.yml file, so builds are green? Or add it first and try to fix the build?

@benasher44
Copy link
Contributor Author

Yep! On master when I checked yesterday, there were 2. There's only the one left after #51. The other one seemed non-trivial to fix.

@benasher44
Copy link
Contributor Author

The final analyzer warning was fixed by #69, which is a pretty awesome patch btw @valeriyvan! This ticket is unblocked now.

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

No branches or pull requests

2 participants