search

nodejs / llnode

An lldb plugin for Node.js and V8, which enables inspection of JavaScript states for insights into Node.js processes and their core dumps.
Other
1.15k stars 100 forks source link

src: apply clang-format rules #276

Closed mmarchini closed 5 years ago

mmarchini commented 5 years ago

Since GitHub/Travis won't rerun tests for Pull Requests when master is updated, some commits landed after https://github.com/nodejs/llnode/commit/85f067aad8737d0d8891fd42b41694415f951a24, which made our linter test fail on `master. Also, since there are only a few open pull requests now, this is unlikely to happen again.

codecov-io commented 5 years ago

Codecov Report

Merging #276 into master will not change coverage. The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #276   +/-   ##
=======================================
  Coverage   81.23%   81.23%           
=======================================
  Files          34       34           
  Lines        4369     4369           
=======================================
  Hits         3549     3549           
  Misses        820      820
Impacted Files Coverage Δ
src/llscan.h 68.18% <ø> (ø) :arrow_up:
src/printer.cc 84.5% <ø> (ø) :arrow_up:
src/llscan.cc 60.73% <66.66%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ecfbdca...53e3c02. Read the comment docs.

mmarchini commented 5 years ago

cc @nodejs/llnode

mmarchini commented 5 years ago

Landed in ecfbdca...62ca523