Skip to content

Conversation

@prashantgupta24
Copy link
Collaborator

Description

Skip_unsupported_compiler_tests

Related Issues

@github-actions
Copy link

👋 Hi! Thank you for contributing to vLLM support on Spyre.
Just a reminder: Make sure that your code passes all the linting checks, otherwise your PR won't be able to be merged. To do so, first install the linting requirements, then run format.sh and commit the changes. This can be done with uv directly:

uv sync --frozen --group lint --active --inexact

Or this can be done with pip:

uv pip compile --group lint > requirements-lint.txt
pip install -r requirements-lint.txt
bash format.sh

Now you are good to go 🚀

Copy link
Collaborator

@joerunde joerunde left a comment

Choose a reason for hiding this comment

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

lpgtm

Signed-off-by: Prashant Gupta <[email protected]>
@prashantgupta24 prashantgupta24 enabled auto-merge (squash) August 18, 2025 21:42
@github-actions github-actions bot added the ready label Aug 18, 2025
@prashantgupta24 prashantgupta24 changed the title Compiler unsupported flag ✅ Compiler unsupported flag for tests Aug 18, 2025
@prashantgupta24
Copy link
Collaborator Author

@yannicks1 the main tests didn't trigger against this PR even after ready label

@prashantgupta24
Copy link
Collaborator Author

bot:test
MARKERS="spyre"

@prashantgupta24 prashantgupta24 merged commit ff48ca8 into main Aug 18, 2025
17 of 18 checks passed
@prashantgupta24 prashantgupta24 deleted the compiler-unsupported-flag branch August 18, 2025 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants