Skip to content

[CI/CD] Pivot to trying a generic webhook#839

Merged
rafvasq merged 1 commit intotorch-spyre:mainfrom
rafvasq:use-webhook
Mar 13, 2026
Merged

[CI/CD] Pivot to trying a generic webhook#839
rafvasq merged 1 commit intotorch-spyre:mainfrom
rafvasq:use-webhook

Conversation

@rafvasq
Copy link
Copy Markdown
Collaborator

@rafvasq rafvasq commented Mar 13, 2026

Description

Related to #825

Checklist

  • I have read the contributing guidelines
  • My code follows the project's code style (run bash format.sh)
  • I have added tests for my changes (if applicable)
  • I have updated the documentation (if applicable)
  • My commits include a Signed-off-by: line (DCO compliance)

Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
@rafvasq rafvasq requested a review from joerunde as a code owner March 13, 2026 17:09
@github-actions
Copy link
Copy Markdown

👋 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, run ./format.sh.
Now you are good to go 🚀.

We also recommend installing prek and configuring it to check your code before every local commit.

Copy link
Copy Markdown
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.

sweet

@rafvasq rafvasq merged commit 4086703 into torch-spyre:main Mar 13, 2026
15 checks passed
joerunde pushed a commit that referenced this pull request Mar 17, 2026
<!-- markdownlint-disable -->

## Related Issues

#825, #839 which don't work due to webhook unable to reach private
Jenkins server.

## Checklist

- [x] I have read the [contributing
guidelines](https://docs.vllm.ai/projects/spyre/en/latest/contributing)
- [x] My code follows the project's code style (run `bash format.sh`)
- [ ] I have added tests for my changes (if applicable)
- [ ] I have updated the documentation (if applicable)
- [x] My commits include a `Signed-off-by:` line (DCO compliance)

Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
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