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

Fix bug 6384 by changing the implementation of locating the assets di... #6386

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ittaigilat-cx
Copy link

…rectory

Closes #6384

Proposed Changes
Instead of looking for the hardwired value "kics" directory in the executable path, the proposed implementation starts from the executable location upwards until it finds the queries directory (which is by default "assets/queries" and can be configured with a command line flag).

I submit this contribution under the Apache-2.0 license.

@github-actions
Copy link
Contributor

kics-logo

KICS version: v1.7.0

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 49
Queries failed to execute placeholder 0
Execution time placeholder 2

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.

assets found in wrong location when having multiple sibling 'kics*' repos with different names
1 participant