Repository navigation
Conversation
Both links in docs/DEVELOPMENT.md pointed at src/skillspector/nodes/llm_analyzer_base.py, a path that never existed in history; the module lives at src/skillspector/llm_analyzer_base.py. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Anton Dziatkovskii <194927794+tonydzi@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
What was broken
docs/DEVELOPMENT.mdlinks tollm_analyzer_base.pytwice (themeta_analyzerrow of the node table, line 134, and the "LLM analyzer base" bullet, line 359). Both links point at../src/skillspector/nodes/llm_analyzer_base.py, and that file is not there, so both links 404 on GitHub.The module is at
src/skillspector/llm_analyzer_base.py, which is whereclass LLMAnalyzerBaseis defined.How I checked
git ls-files | grep llm_analyzer_basereturns onlysrc/skillspector/llm_analyzer_base.py.gh api "repos/NVIDIA/SkillSpector/commits?path=src/skillspector/nodes/llm_analyzer_base.py" --jq lengthreturns0. Thenodes/path was never in the history, so this is a wrong path in the doc rather than a moved file.docs/andcontrib/, and these two are the only broken ones.What changed
Both links now point to
../src/skillspector/llm_analyzer_base.py. The neighbouringmeta_analyzer.pylink in the same table cell was already correct and is untouched. Nothing else in the file changed.Anton Dziatkovskii · github.com/tonydzi