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

Run linter (clang-format) on Travis #260

Closed mmarchini closed 5 years ago

joyeecheung commented 5 years ago

Ping @mmarchini This needs a rebase

codecov-io commented 5 years ago

Codecov Report

Merging #260 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #260   +/-   ##
=======================================
  Coverage   79.96%   79.96%           
=======================================
  Files          34       34           
  Lines        4292     4292           
=======================================
  Hits         3432     3432           
  Misses        860      860
Impacted Files Coverage Δ
src/llv8-constants.cc 85.61% <ø> (ø) :arrow_up:
src/llv8.cc 76.04% <ø> (ø) :arrow_up:
src/llnode_api.h 0% <ø> (ø) :arrow_up:
src/llscan.cc 55.26% <100%> (ø) :arrow_up:
src/llnode.cc 77.31% <100%> (ø) :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 ea06ef5...bfc7d64. Read the comment docs.

mmarchini commented 5 years ago

Done. I also moved clang-format dependency to devDependencies.

mmarchini commented 5 years ago

Landed in ea06ef5...85f067a