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

Deepscan Tool Integration with Git Flow #40

Closed
wants to merge 2 commits into from
Closed

Conversation

joeni21
Copy link

@joeni21 joeni21 commented Oct 23, 2024

Integrated DeepScan Tool with our current github workflow, as a dynamic tool that can help us with our development in the future. It is connected via API and will Run DeepScan upon each pull request.

Below is a screenshot from the report that is linked to our GitHub Repository.
Screenshot 2024-10-24 at 10 20 40 AM

Also, do note that within this pull request one can see that after the Lint and Test, we can see that DeepScan is displayed in "All Checks Have Passed".

@joeni21 joeni21 added the enhancement New feature or request label Oct 23, 2024
@joeni21 joeni21 self-assigned this Oct 23, 2024
@coveralls
Copy link

coveralls commented Oct 23, 2024

Pull Request Test Coverage Report for Build 11489313973

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.005%) to 82.712%

Files with Coverage Reduction New Missed Lines %
src/user/approval.js 3 87.61%
Totals Coverage Status
Change from base Build 11488777020: -0.005%
Covered Lines: 22357
Relevant Lines: 25612

💛 - Coveralls

@LeeMEow23
Copy link

We decided not to use this tool, will close this PR.

@LeeMEow23 LeeMEow23 closed this Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants