Page MenuHomePhabricator

Added guide for OSX users on how to install clang-format-8.
ClosedPublic

Authored by tobias_ruck on Wed, Jan 8, 15:35.

Details

Reviewers
deadalnix
Fabien
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABC55577b249220: Added guide for OSX users on how to install clang-format-8.
Summary

Added the command that allowed me to use clang-format-8 without having to recompile llvm & clang to CONTRIBUTING.md.

Test Plan

I ran npm install -g clang-format@1.2.4. I had to delete/move some files due to conflicts with another installation, but after that, running arc diff succeeded. A fresh installation should work with just that command; otherwise the instructions can be amended to accomodate for additional problems.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

tobias_ruck created this revision.Wed, Jan 8, 15:35
Owners added a reviewer: Restricted Owners Package.Wed, Jan 8, 15:35
Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, Jan 8, 15:35
Fabien accepted this revision.Wed, Jan 8, 16:34
Fabien added a subscriber: Fabien.

Tested working on Catalina.

This revision is now accepted and ready to land.Wed, Jan 8, 16:34
tobias_ruck edited the test plan for this revision. (Show Details)Thu, Jan 9, 00:11