Skip to content

Update clang-format version to latest on Travis #2115

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

Closed
wants to merge 1 commit into from
Closed

Conversation

bstpierr
Copy link
Contributor

Homebrew is now serving up clang-format v8.

Homebrew is now serving up clang-format v8.
@bstpierr
Copy link
Contributor Author

It actually looks like some of the style rules have changed between version 7 and 8. See this commit on my other PR. clang-format 8 had fixes the spacing and long line, but clang-format 7 saw those as wrong.

Do we have a plan for how to handle style discrepancies between clang-format versions?

@ryanwilson
Copy link
Member

It looks like Travis is still pulling from 7: https://travis-ci.org/firebase/firebase-ios-sdk/jobs/458598973#L684

Let's set up a discussion to see what the strategy is, if we should always move to latest or not.

@bstpierr
Copy link
Contributor Author

From the documentation it seems like they just use what's on their disk image, but it can run brew update if we would like. In the past we just allowed it to stay old, but update the script like I did in this PR. This allows us to update the version on our machines ahead of Travis and fix style issues before Travis breaks us.

@bstpierr bstpierr closed this Nov 26, 2018
@bstpierr
Copy link
Contributor Author

Oops, didn't mean to close this PR. Reopening it.

@bstpierr bstpierr reopened this Nov 26, 2018
Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update instructions at https://github.com/firebase/firebase-ios-sdk#code-formatting

We should probably wait until after M40 for bumping the clang-format version since likely it would force all of open source to release.

@bstpierr bstpierr added this to the M41 milestone Dec 6, 2018
@bstpierr bstpierr self-assigned this Dec 6, 2018
@bstpierr bstpierr changed the title Bump style.sh version Update clang-format version to latest on Travis Dec 6, 2018
@paulb777
Copy link
Member

paulb777 commented Jan 2, 2019

Updating clang-format turned out to be a natural follow-on of updating travis to Xcode 10.1 and I did it in #2222.

@paulb777 paulb777 closed this Jan 2, 2019
@paulb777 paulb777 deleted the bs-clangver branch January 2, 2019 22:50
@firebase firebase locked and limited conversation to collaborators Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants