Skip to content

Conversation

@bhimrazy
Copy link
Collaborator

@bhimrazy bhimrazy commented Jan 7, 2026

What does this PR do?

This PR dynamically fetches the transformers version from the Lightning Thunder requirements instead of hardcoding it, improving maintainability and keeping CI dependency versions in sync.

Follow up to #2149

…ments

- Replace hardcoded transformers==4.52.4 with dynamic version fetching
- Fetch version from thunder's test requirements file
- Improve CI maintainability by keeping versions in sync
@bhimrazy bhimrazy marked this pull request as ready for review January 8, 2026 06:55
@bhimrazy bhimrazy requested review from lantiga and t-vi as code owners January 8, 2026 06:55
Copilot AI review requested due to automatic review settings January 8, 2026 06:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves CI maintainability by dynamically fetching the transformers version from Lightning Thunder's requirements file instead of hardcoding it. This ensures the test dependency versions stay synchronized with what Lightning Thunder expects.

Key Changes:

  • Replaced hardcoded transformers==4.52.4 with dynamic version extraction via curl
  • Added shell script pipeline to parse the version from Lightning Thunder's test requirements

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants