Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Removed restriction for folder name 'app' inside the source code under test. #13

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

sa-ny
Copy link
Contributor

@sa-ny sa-ny commented Oct 14, 2024

  1. BatchFix and SingleFix now each have their own separate list of supported CWEs.
  2. The process for creating the source code tar file and flaw.json for BatchFix has been moved out of the source code folder to temp folder.

@sa-ny sa-ny requested a review from julz0815 October 14, 2024 08:22
@sa-ny sa-ny changed the title App foldername change Removed restriction for folder name 'app' inside the source code under test. Oct 16, 2024
@sa-ny sa-ny marked this pull request as draft October 16, 2024 07:08
@sa-ny sa-ny marked this pull request as ready for review October 18, 2024 05:19
@@ -0,0 +1,28 @@
export const sourcecodeFolderName = 'app/';
//export temp folder value from github action
export const tempFolder = process.env.RUNNER_TEMP ? (process.env.RUNNER_TEMP + '/') : '';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we sure the tempfolder is writeable in all runner images a customer may use? I would not make this hardcoded to temp, maybe as a default, but have it configurable maybe?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants