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

Add option to scan only certain directory/paths instead of the whole repo 📁 #283

Closed
jamacku opened this issue Aug 8, 2023 · 2 comments · Fixed by #294
Closed

Add option to scan only certain directory/paths instead of the whole repo 📁 #283

jamacku opened this issue Aug 8, 2023 · 2 comments · Fixed by #294
Assignees
Labels
Milestone

Comments

@jamacku
Copy link
Member

jamacku commented Aug 8, 2023

Type of issue

Feature Request

Description

Imagine the following repo structure:

src/
scripts/
  1.sh
  2.sh
  text.txt
  main.py
  ...
doc/
...

It would be great to have the option to specify certain directories on which the autodetection of ShellScripts should be performed.

Describe the solution you'd like

Introduce new input, e.g., scan-path or script-path, and implement the abovementioned functionality.

@jamacku jamacku added the type: feature New feature label Aug 8, 2023
@jamacku jamacku self-assigned this Aug 8, 2023
@jamacku jamacku added this to the v5.0.0 milestone Aug 10, 2023
@jamacku
Copy link
Member Author

jamacku commented Aug 10, 2023

Using cd isn't recommended since we need a full path of scanned files for SARIF.

The best approach is to filter file paths based on provided input.

@jamacku
Copy link
Member Author

jamacku commented Aug 11, 2023

This feature could enable us to implement some e2e tests.

@jamacku jamacku pinned this issue Aug 11, 2023
@jamacku jamacku linked a pull request Aug 14, 2023 that will close this issue
4 tasks
@jamacku jamacku unpinned this issue Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant