Page MenuHomePhabricator
Differential D6389

Simplify install instructions for linter dependencies
ClosedPublic
Actions

Authored by jasonbcox on Jun 5 2020, 18:49.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC883815514dfe: Simplify install instructions for linter dependencies
Summary

This patch makes this section of the doc more consistent
and less repetitive. It also reduces the number of copy-paste
install steps from two to one on recent Ubuntu and Debian distros.

Test Plan

Read it.

Diff Detail

Event Timeline

jasonbcox created this revision.Jun 5 2020, 18:49
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 5 2020, 18:49
jasonbcox requested review of this revision.Jun 5 2020, 18:49
Harbormaster completed remote builds in B11231: Diff 20952.Jun 5 2020, 19:02
Fabien accepted this revision.Jun 5 2020, 19:15
Fabien added a subscriber: Fabien.
Fabien added inline comments.
CONTRIBUTING.md
137

We really want clang-format-8

This revision is now accepted and ready to land.Jun 5 2020, 19:15
jasonbcox retitled this revision from Simply install instructions for linter depenendencies to Simply install instructions for linter dependencies.Jun 5 2020, 19:30
jasonbcox retitled this revision from Simply install instructions for linter dependencies to Simplify install instructions for linter dependencies.Jun 5 2020, 20:28
jasonbcox updated this revision to Diff 20959.Jun 5 2020, 20:29

clang-format -> clang-format-8

Harbormaster completed remote builds in B11238: Diff 20959.Jun 5 2020, 20:39
Closed by commit rABC883815514dfe: Simplify install instructions for linter dependencies (authored by jasonbcox). · Explain WhyJun 5 2020, 21:07
This revision was automatically updated to reflect the committed changes.
jasonbcox added a commit: rABC883815514dfe: Simplify install instructions for linter dependencies.

Revision Contents
Changeset List

PathSize
17 lines

Diff 20952

View Options

CONTRIBUTING.md

Loading...