Page MenuHomePhabricator

[yarn] Bump `clang-format` to `1.7.0`
ClosedPublic

Authored by atul on Apr 27 2022, 8:12 AM.
Tags
None
Referenced Files
F2776871: D3859.diff
Fri, Sep 20, 6:26 AM
F2765465: D3859.diff
Thu, Sep 19, 2:42 PM
Unknown Object (File)
Mon, Sep 16, 1:18 PM
Unknown Object (File)
Mon, Sep 16, 1:07 PM
Unknown Object (File)
Mon, Sep 9, 5:43 PM
Unknown Object (File)
Sun, Aug 25, 1:39 PM
Unknown Object (File)
Sun, Aug 25, 12:34 AM
Unknown Object (File)
Aug 18 2024, 7:46 AM

Details

Summary

Downstream dependencies have issues surfaced by yarn audit

The repo doesn't have release notes (that I could find), but looking at the commits since 1.5.0 they're all bumping deps and messing with CI


Depends on D3858

Test Plan

CI

Diff Detail

Repository
rCOMM Comm
Branch
landapr27 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul requested review of this revision.Apr 27 2022, 8:19 AM

Please make sure to make me a blocking reviewer on diffs that affect dependencies in the future

At some point in the future we should move this to Nix

This revision is now accepted and ready to land.Apr 27 2022, 2:32 PM

Please make sure to make me a blocking reviewer on diffs that affect dependencies in the future

Will do

At some point in the future we should move this to Nix

As in include clang-format binary directly instead of sourcing from npm package?

In D3859#108122, @atul wrote:

At some point in the future we should move this to Nix

As in include clang-format binary directly instead of sourcing from npm package?

Yeah

This revision was automatically updated to reflect the committed changes.