Skip to content
Open
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
f427a8b
feat: Add automated dependency version tracking and extraction
Oct 10, 2025
7ea0b89
Add SPDX copyright header to extraction script
Oct 10, 2025
62b6e1f
Fix pre-commit issues: import order and EOF
Oct 10, 2025
b283998
Remove unintended files and update .gitignore
Oct 10, 2025
692b308
Fix pre-commit issues: executable permission and unused variables
Oct 10, 2025
318f45c
Fix remaining unused variables: service and critical_reason
Oct 10, 2025
51af209
Make baseline dependency count dynamic based on previous extraction
Oct 10, 2025
749d8e2
Add removed dependencies tracking and PR reporting
dagil-nvidia Oct 10, 2025
cc3c88f
Fix YAML syntax error in nightly workflow
dagil-nvidia Oct 10, 2025
d520194
Fix all pre-commit issues: YAML syntax and formatting
dagil-nvidia Oct 10, 2025
9ac63de
Fix shell script extraction to skip runtime-determined versions
dagil-nvidia Oct 10, 2025
c0722d0
feat: Enhance dependency extraction to capture pip installs and binar…
dagil-nvidia Oct 13, 2025
3b0e8ab
fix: Remove trailing whitespace from extraction script
dagil-nvidia Oct 13, 2025
bd37921
fix: address CodeRabbit and reviewer feedback
dagil-nvidia Oct 14, 2025
e4619f1
Address nv-tusharma's review comments
dagil-nvidia Oct 17, 2025
f5a8770
feat(deps): add version discrepancy detection and composite action
dagil-nvidia Oct 20, 2025
3b5a644
fix: apply black and ruff formatting fixes
dagil-nvidia Oct 20, 2025
5ff3749
fix(deps): improve discrepancy detection accuracy
dagil-nvidia Oct 20, 2025
f547bae
fix(deps): skip sub-dependency ARGs, normalize pinning, and document …
dagil-nvidia Oct 20, 2025
a1860f0
fix: apply isort and black formatting
dagil-nvidia Oct 20, 2025
3972458
feat(deps): add automated failure monitoring for dependency extractio…
dagil-nvidia Oct 20, 2025
a86a3f5
fix: address pre-commit trailing whitespace issues
dagil-nvidia Oct 21, 2025
e91ff3e
refactor(deps): address nv-anants review feedback
dagil-nvidia Oct 21, 2025
acc4450
docs(deps): add comprehensive documentation for dependency extraction
dagil-nvidia Oct 21, 2025
b98d2cf
chore: trigger DCO re-check
dagil-nvidia Oct 21, 2025
f041e40
refactor(deps): modularize dependency extraction system
dagil-nvidia Oct 22, 2025
32456dc
feat(deps): add extractor architecture and unit tests
dagil-nvidia Oct 22, 2025
512c347
Merge branch 'main' into feature/dependency-extraction-DYN-1235
dagil-nvidia Oct 22, 2025
c163fa4
fix: apply pre-commit formatting fixes and update test markers
dagil-nvidia Oct 22, 2025
d20b575
feat(deps): add dynamic FRAMEWORK_VERSIONS.md generator
dagil-nvidia Oct 22, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add SPDX copyright header to extraction script
Signed-off-by: Dan Gil <[email protected]>
  • Loading branch information
Dan Gil authored and dagil-nvidia committed Oct 21, 2025
commit 7ea0b892a6375ff7a89e6ac3807badc2802f5a1a
15 changes: 15 additions & 0 deletions .github/workflows/extract_dependency_versions.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,19 @@
#!/usr/bin/env python3
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Shebang requires executable bit or remove it

Pre-commit flagged: has a shebang but not executable. Either chmod +x this file in git or drop the shebang (workflows call it via python3).

Suggested options:

  • git update-index --chmod=+x .github/workflows/extract_dependency_versions.py
  • Or remove line 1 (shebang).
🧰 Tools
πŸͺ› GitHub Actions: Pre Merge Validation of (ai-dynamo/dynamo/refs/pull/3547/merge) by dagil-nvidia.

[error] 1-1: isort: Files were modified by this hook. (pre-commit)


[error] 1-1: Black: 1 file reformatted. (pre-commit)


[error] 1-1: check-shebang-scripts-are-executable: has a shebang but is not marked executable.

πŸͺ› Ruff (0.13.3)

1-1: Shebang is present but file is not executable

(EXE001)

πŸ€– Prompt for AI Agents
.github/workflows/extract_dependency_versions.py around line 1: the file
contains a shebang but is not executable which pre-commit flagged; fix by either
making the file executable in git (run: git update-index --chmod=+x
.github/workflows/extract_dependency_versions.py and commit that change) or
remove the shebang line (delete the first line and commit), choose one approach
and commit the resulting change so pre-commit no longer fails.

# SPDX-FileCopyrightText: Copyright (c) 2024-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
# SPDX-License-Identifier: Apache-2.0
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

"""
Extract all dependency versions from Dockerfiles and requirements files.
Generates a CSV file with all dependencies across trtllm, vllm, sglang, and operator components.
Expand Down