Skip to content

Conversation

@data-douser
Copy link
Collaborator

Modifies the cds-compilation-for-actions.test.sh script in order to:

  • avoid duplicate work in compiling to model.cds.json per project;
  • ensure that javascript/frameworks/cap/test/models/** projects are considered in-scope for generating model.cds.json file per project;
  • avoid attempting CDS compiliation tasks for test/project directories that do not contain any .cds files at any supportable project path.

Modifies the `cds-compilation-for-actions.test.sh` script in order to:
 * avoid duplicate work in compiling to `model.cds.json` per project;
 * ensure that `javascript/frameworks/cap/test/models/**` projects are
   considered in-scope for generating `model.cds.json` file per project;
 * avoid attempting CDS compiliation tasks for test/project directories
   that do not contain any `.cds` files at any supportable project path.
Migrates `*.test.sh` scripts from `extractors/cds/tools/test/` dir
to `extractors/cds/tools/workflow/` dir and changes script file
extensions from `.test.sh` to just `.sh`.
Copy link
Contributor

@jeongsoolee09 jeongsoolee09 left a comment

Choose a reason for hiding this comment

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

Just a minor suggestion.

@data-douser data-douser changed the base branch from jeongsoolee09/restrict-cap-remoteflowsource-properties to main August 4, 2025 21:31
@data-douser data-douser changed the base branch from main to jeongsoolee09/restrict-cap-remoteflowsource-properties August 4, 2025 22:12
@data-douser
Copy link
Collaborator Author

Closing this PR in favor of merging the equivalent set of changes to main via #212 .

@data-douser data-douser closed this Aug 5, 2025
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