Skip to content

Conversation

jzolnowski
Copy link

@jzolnowski jzolnowski commented Jul 26, 2023

  • exclude .md and .txt files from analysis
  • upgrade codeql to v3 - the current version of the CodeQL (v1) was deprecated on January 18th, 2023, and is no longer updated or supported. For better performance, improved security, and new features, we upgrade it to v3. [More information](https://github.blog/changelog/2023-01-18-code-scanning-codeql-action-v1-is-now-deprecated
  • comment out the autobuild step, which is not needed with javascript-based code
  • set the timeout-minutes option to 360 - the maximum number of minutes to run the step before killing the process

@jzolnowski jzolnowski requested review from Splaktar and gkalpak July 26, 2023 10:16
@gkalpak
Copy link

gkalpak commented Jul 26, 2023

Let's put this on hold, until we finalize the proper configuration/approach in xlts-dev/angularjs-webpack-babel#29.

@gkalpak gkalpak marked this pull request as draft July 26, 2023 12:02
- exclude .md and .txt files from analysis
- upgrade codeql to `v3` - the current version of the CodeQL (`v1`)
  was deprecated on January 18th, 2023, and is no longer updated or
  supported. For better performance, improved security, and new
  features, we upgrade it to v3. [More
  information](https://github.blog/changelog/2023-01-18-code-scanning-codeql-action-v1-is-now-deprecated)
- comment out the autobuild step, which is not needed with javascript-based code
- set the timeout-minutes option to 360 - the maximum number of minutes to run the step before killing the process
@jzolnowski jzolnowski force-pushed the improve-code-scanning-config branch from 11b89b5 to 56e59ea Compare July 31, 2023 18:44
@jzolnowski jzolnowski marked this pull request as ready for review July 31, 2023 18:44
@gkalpak gkalpak merged commit 65046e1 into master Aug 1, 2023
@gkalpak gkalpak deleted the improve-code-scanning-config branch August 1, 2023 18:12
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.

2 participants