Change Details

`llvm` 8.0 and 7.1 have been released recently. As an example, Ubuntu recently updated to clang-format 7.1, causing arcanist to return an error on `arc lint` due to version conflict. I tested our codebase against clang-format 7.1 and 8.0, there is a minor difference in the output (some edge cases are causing bad indents with clang-format-7, which is fixed by clang-format-8). Therefore both versions cannot be allowed at the same time as it would cause `arc lint` to format back and forth when using different versions. I would have preferred to enforce >= 7.0 and < 8 but this is not supported by arcanist. As a workaround an additional check is performed at the linter level to enforce using version 7.x only.

`llvm` 8.0 and 7.1 have been released recently. As an example, Ubuntu recently updated to clang-format 7.1, causing arcanist to return an error on `arc lint` due to version conflict. I tested our codebase against clang-format 7.1 and 8.0, there is a minor difference in the output (some edge cases are causing bad indents with clang-format-7, which is fixed by clang-format-8). Therefore both versions cannot be allowed at the same time as it would cause `arc lint` to format back and forth when using different versions. I would have preferred to enforce >= 7.0 and < 8 in the `.arclint`file but this is not supported by arcanist. As a workaround an additional check is performed at the linter level to enforce using version 7.x only.

`llvm` 8.0 and 7.1 have been released recently. As an example, Ubuntu recently updated to clang-format 7.1, causing arcanist to return an error on `arc lint` due to version conflict. I tested our codebase against clang-format 7.1 and 8.0, there is a minor difference in the output (some edge cases are causing bad indents with clang-format-7, which is fixed by clang-format-8). Therefore both versions cannot be allowed at the same time as it would cause `arc lint` to format back and forth when using different versions. I would have preferred to enforce >= 7.0 and < 8 but this is notin the supported by arcanist. As a workaround an additional check is`.arclint`file but this is not supported by arcanist. As a workaround an additional check is performed at the linter level to enforce using version 7.x only.