Skip to content

Move html template source location to raw github#629

Merged
sjmonson merged 1 commit intomainfrom
fix/html_url
Mar 12, 2026
Merged

Move html template source location to raw github#629
sjmonson merged 1 commit intomainfrom
fix/html_url

Conversation

@sjmonson
Copy link
Collaborator

Summary

Point UI to HTML template on raw.githubusercontent.com rather than blog.vllm.ai.

Test Plan

Will have to be tested after CI fires.

Related Issues


  • "I certify that all code in this PR is my own, except as noted below."

Use of AI

  • Includes AI-assisted code completion
  • Includes code generated by an AI application
  • Includes AI-generated tests (NOTE: AI written tests should have a docstring that includes ## WRITTEN BY AI ##)

Signed-off-by: Samuel Monson <smonson@redhat.com>
@github-actions
Copy link

github-actions bot commented Mar 11, 2026

🧹 The live preview for this PR has been removed.

github-actions bot pushed a commit that referenced this pull request Mar 11, 2026
@sjmonson sjmonson requested review from DaltheCow and dbutenhof March 11, 2026 19:08
Copy link
Collaborator

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

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

OK, that seems to work. (That is, without researching exactly what ought to be there, I can find index.html and manifest.json.)

I note that the URL seems to work with or without your /refs/head phrase: e.g., https://raw.githubusercontent.com/vllm-project/guidellm/gh-pages/ui/pr/629/manifest.json.

@sjmonson
Copy link
Collaborator Author

Merging this now since it works. We can clean up the URLs later.

@sjmonson sjmonson merged commit a9c8d92 into main Mar 12, 2026
16 checks passed
@sjmonson sjmonson deleted the fix/html_url branch March 12, 2026 00:44
github-actions bot pushed a commit that referenced this pull request Mar 12, 2026
@dbutenhof
Copy link
Collaborator

Merging this now since it works. We can clean up the URLs later.

And /dev and /nightly are live now, too. This looks great, and aside from nit-picking the URL suffixes, I like the idea of referring to GitHub rather than putting it somewhere else.

sjmonson added a commit that referenced this pull request Mar 12, 2026
## Summary

Point UI to HTML template on `raw.githubusercontent.com` rather than
`blog.vllm.ai`.

## Test Plan

Will have to be tested after CI fires.

## Related Issues

- Resolves #627 

---

- [x] "I certify that all code in this PR is my own, except as noted
below."

## Use of AI

- [ ] Includes AI-assisted code completion
- [ ] Includes code generated by an AI application
- [ ] Includes AI-generated tests (NOTE: AI written tests should have a
docstring that includes `## WRITTEN BY AI ##`)
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.

HTML report render and generation fail, possibly caused by latest Github pages build error

2 participants