feat: Enable code scanning #354
Conversation
| matrix: | ||
| language: [ 'javascript', 'ruby' ] | ||
| # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python', 'ruby' ] | ||
| # Learn more about CodeQL language support at https://git.io/codeql-language-support |
There was a problem hiding this comment.
Is there any advantage to running both of these individually as opposed to just specifying all relevant languages below in the initialize step?
| # Autobuild attempts to build any compiled languages (C/C++, C#, or Java). | ||
| # If this step fails, then you should remove it and run the build manually (see below) | ||
| - name: Autobuild | ||
| uses: github/codeql-action/autobuild@v1 |
There was a problem hiding this comment.
There isn't really anything to build here, can this step be skipped if it'll save time?
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
adzuci
force-pushed
the
adzuci/ruby-code-scanning
branch
from
48a8862 to
113e0bd
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #354 +/- ##
=======================================
Coverage 96.16% 96.16%
=======================================
Files 58 58
Lines 4588 4588
=======================================
Hits 4412 4412
Misses 176 176 ☔ View full report in Codecov by Sentry. |
I noticed code scanning now supports Ruby, so I'm putting up this PR for review.