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.

Sign up for GitHub

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

Jump to bottom

feat(analyzer): add expected lines of code in analyzer #6222

Merged
merged 7 commits into from
Mar 15, 2023
Merged

feat(analyzer): add expected lines of code in analyzer #6222

merged 7 commits into from
Mar 15, 2023

Conversation

cxMiguelSilva
Copy link
Collaborator

Proposed Changes

  • add expected loc in the analyzer
  • add line_counter to utils
  • add/update tests

I submit this contribution under the Apache-2.0 license.

@cxMiguelSilva cxMiguelSilva added go Pull requests that update Go code tests Automated testing labels Mar 8, 2023
@cxMiguelSilva cxMiguelSilva self-assigned this Mar 8, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023 •
edited
Loading

kics-logo

KICS version: v1.6.6

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 49
Queries failed to execute placeholder 0
Execution time placeholder 3

github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 8, 2023
View reviewed changes
pkg/utils/line_counter.go Fixed Show fixed Hide fixed
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 8, 2023
View reviewed changes
pkg/utils/line_counter.go Fixed Show fixed Hide fixed
pkg/utils/line_counter.go Fixed Show fixed Hide fixed
@cxMiguelSilva cxMiguelSilva merged commit ccacecb into master Mar 15, 2023
@cxMiguelSilva cxMiguelSilva deleted the feat_add-expected_loc_analyzer branch March 15, 2023 15:15
@chenrui333 chenrui333 mentioned this pull request Mar 28, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joaoReigota1 joaoReigota1 joaoReigota1 approved these changes

@gabriel-cx gabriel-cx Awaiting requested review from gabriel-cx

Assignees

@cxMiguelSilva cxMiguelSilva

Labels
go Pull requests that update Go code tests Automated testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cxMiguelSilva @joaoReigota1