-
-
Notifications
You must be signed in to change notification settings - Fork 551
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
ScanCode contains too many data files #3049
Comments
The first action seems to be the simplest solution. |
I think YAML front-matter would be much better. There are a lot of issues in YAML when it comes to text with whitespace/symbols as we were discussing, and plain YAMLs are causing a lot of YAML read errors. |
* Delete all .yml files for rules * Modify .RULE files to contain their data as YAML frontmatter Reference: #3049 Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
This commit ends the process of merging .RULE and .yml files into a single .RULE file which has YAML frontmatter storing the rule metadata present in the .yml file previously. This renaming and merging has been done to preserve line-history for both the files. Reference: #3049 Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
are we going ahead with this? Then the options are:
|
IMHO let's wait a bit for this |
|
The src/licensedcode/data directory contains 68K+ files and 64k just for the rules.
These rule files are not used much at runtime because they are baked into the index in a compressed form that is used at runtime. The same applies to the licenses files that are fully included in the index in an object form.
These are only needed when the index is rebuilt.
Another issue is that handling so many files makes any filesystem operation (unbearably) slow including during development time and at installation time.
It also creates side issues as #2427 (comment) and linkedin/shiv#224
I suggest some of these to fix the issue:
Combining either these three actions or just the last two should make this OK and workable both for development, installation and runtime.
The text was updated successfully, but these errors were encountered: