Bugfix/clean unreachable #503
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Use a sorted list of files so the output is more deterministic. Purely cosmetic. True randomness would actually improve our search times here. Signed-off-by: Randolph Sapp <[email protected]>
Keep the full file paths around and expand them after the search. This search is still fairly rudimentary, but right now these are hard confirmations on potentially multiple files for a single name. Signed-off-by: Randolph Sapp <[email protected]>
Add a convenient delete function to strip confirmed unused files from the current source directory. It is not set by default and, while it is very specific about the directory it's searching and the files it finds, I advise you use it with caution. Signed-off-by: Randolph Sapp <[email protected]>
These files aren't needed anymore. Detected and removed with check_files.py. Signed-off-by: Randolph Sapp <[email protected]>
StaticRocket
requested review from
VeeruPrudhvi,
aniket-l,
cshilwant,
praneethbajjuri and
uditkumarti
as code owners
This was referenced Oct 25, 2025
praneethbajjuri
approved these changes
Oct 28, 2025
There was a problem hiding this comment.
@vishalmti can you review the PLL release specific doc pruning. Are we no longer referring this in any release docs ?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix(unused): prune currently unused files
These files aren't needed anymore. Detected and removed with check_files.py.
Signed-off-by: Randolph Sapp [email protected]
ci(check_files): add delete flag
Add a convenient delete function to strip confirmed unused files from the
current source directory. It is not set by default and, while it is very
specific about the directory it's searching and the files it finds, I advise
you use it with caution.
Signed-off-by: Randolph Sapp [email protected]
ci(check_files): expand file paths after search
Keep the full file paths around and expand them after the search. This search
is still fairly rudimentary, but right now these are hard confirmations on
potentially multiple files for a single name.
Signed-off-by: Randolph Sapp [email protected]
ci(check_files): use sorted list of files
Use a sorted list of files so the output is more deterministic. Purely
cosmetic. True randomness would actually improve our search times here.
Signed-off-by: Randolph Sapp [email protected]