Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(static analysis): repurposing lizard.yml to host all the static analysis tools to run on PRs #444

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

SuperFola
Copy link
Member

Description

Adds cppcheck to the CI, running on PRs (when they are created and modified I hope)

Checklist

  • I have read the Contributor guide
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation if needed
  • I have added tests that prove my fix/feature is working
  • New and existing tests pass locally with my changes

@SuperFola SuperFola force-pushed the ci/static-analysis branch from b68a618 to c1229f8 Compare March 25, 2024 21:02
@SuperFola SuperFola force-pushed the ci/static-analysis branch from c1229f8 to bb73989 Compare March 25, 2024 21:04
Copy link

Lizard report

Listing only functions with cyclomatic complexity >= 15 or NLOC >= 100 or parameters >= 10.

Filename Start line:end line Function name Parameters NLOC CCN
./src/arkreactor/VM/VM.cpp 258:1153 Ark::VM::safeRun 2 644 165
./src/arkreactor/Compiler/BytecodeReader.cpp 70:438 Ark::BytecodeReader::display 4 327 131
./src/arkreactor/Compiler/Macros/Processor.cpp 219:463 Ark::internal::MacroProcessor::evaluate 2 206 72
./src/arkscript/main.cpp 19:295 main 2 237 29
./src/arkreactor/Compiler/Compiler.cpp 530:635 Ark::Compiler::handleCalls 5 78 27
./src/arkreactor/Compiler/Compiler.cpp 261:352 Ark::Compiler::compileExpression 5 74 24
./include/Ark/Compiler/AST/Parser.hpp 71:149 Ark::internal::ARK_APIParser::string 0 77 23
./src/arkreactor/Compiler/AST/Node.cpp 183:253 Ark::internal::operator < < 2 61 23
./src/arkreactor/Compiler/JsonCompiler.cpp 25:190 Ark::JsonCompiler::_compile 1 139 23
./src/arkreactor/VM/State.cpp 148:288 Ark::State::configure 0 116 22
./include/utf8.hpp 137:183 utf8::isValid 1 44 21
./src/arkreactor/Compiler/Macros/Processor.cpp 121:185 Ark::internal::MacroProcessor::processNode 2 48 20
./src/arkreactor/Compiler/Macros/Executors/Function.cpp 12:87 Ark::internal::FunctionExecutor::applyMacro 1 61 18
./src/arkreactor/VM/Value.cpp 146:225 Ark::Value::toString 2 67 17
./src/arkreactor/Compiler/AST/Parser.cpp 234:325 Ark::internal::Parser::import_ 0 72 17
./src/arkreactor/Compiler/AST/Node.cpp 117:181 Ark::internal::Node::repr 0 56 17
./include/Ark/Compiler/AST/Predicates.hpp 162:186 Ark::internal::IsSymbol::operator ( ) 1 24 16
./src/arkscript/REPL/Repl.cpp 14:106 Ark::Repl::run 0 74 16
./src/arkreactor/Compiler/Macros/Processor.cpp 53:101 Ark::internal::MacroProcessor::registerMacro 1 44 16
./src/arkreactor/TypeChecker.cpp 27:78 Ark::types::displayContract 2 45 15

CppCheck report


@ArkScript-lang ArkScript-lang deleted a comment from github-actions bot Mar 25, 2024
@SuperFola SuperFola merged commit 6652413 into dev Mar 25, 2024
16 checks passed
@SuperFola SuperFola deleted the ci/static-analysis branch March 25, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant