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

PyDriller for explore git repo #424

Closed
wants to merge 1 commit into from
Closed

Conversation

babenek
Copy link
Contributor

@babenek babenek commented Sep 21, 2023

Description

Please include a summary of the change and which is fixed.

review after #433

  • Add CMD arguments for scanning git repositories for every commit changed files

How has this been tested?

Please describe the tests that you ran to verify your changes.

  • UnitTest
  • clone any git repo and scan

@babenek babenek force-pushed the pydriller branch 4 times, most recently from 80f8368 to 4fcdf66 Compare September 25, 2023 12:58
@codecov-commenter
Copy link

codecov-commenter commented Sep 25, 2023

Codecov Report

Attention: 48 lines in your changes are missing coverage. Please review.

Comparison is base (6b78cf2) 90.73% compared to head (7ff1415) 89.61%.
Report is 2 commits behind head on main.

Files Patch % Lines
credsweeper/__main__.py 43.93% 34 Missing and 3 partials ⚠️
credsweeper/scanner/scan_type/pem_key_pattern.py 80.00% 4 Missing and 4 partials ⚠️
credsweeper/filters/value_jfrog_token_check.py 88.88% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #424      +/-   ##
==========================================
- Coverage   90.73%   89.61%   -1.12%     
==========================================
  Files         125      125              
  Lines        4252     4364     +112     
  Branches      674      699      +25     
==========================================
+ Hits         3858     3911      +53     
- Misses        261      311      +50     
- Partials      133      142       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@babenek babenek force-pushed the pydriller branch 4 times, most recently from 45900b2 to 349c669 Compare October 8, 2023 09:43
@babenek babenek marked this pull request as ready for review October 8, 2023 09:57
@babenek babenek requested a review from a team as a code owner October 8, 2023 09:57
Yullia
Yullia previously approved these changes Oct 9, 2023
@babenek babenek marked this pull request as draft October 10, 2023 07:57
@babenek babenek mentioned this pull request Oct 16, 2023
2 tasks
@babenek babenek force-pushed the pydriller branch 2 times, most recently from f818c2a to 5fd76e5 Compare October 16, 2023 11:00
@babenek babenek requested a review from Dmitriy-NK October 17, 2023 11:05
@babenek babenek force-pushed the pydriller branch 2 times, most recently from 03a3ae8 to 16d0c68 Compare October 21, 2023 11:44
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.

3 participants