Skip to content

Conversation

aireilly
Copy link
Collaborator

@aireilly aireilly commented Jul 18, 2025

SUMMARY:

  • Adds a simple docs Makefile
  • Cleans up the docs front page

@aireilly aireilly marked this pull request as draft July 18, 2025 14:06
Copy link

👋 Hi! Thank you for contributing to llm-compressor. Please add the ready label when the PR is ready for review.

Note: This is required to complete the testing suite, please only add the label once the PR is code complete and local testing has been performed.

gemini-code-assist[bot]

This comment was marked as outdated.

gemini-code-assist[bot]

This comment was marked as outdated.

@aireilly aireilly changed the title Clean up docs build [Docs] Clean up docs build Jul 18, 2025
@aireilly aireilly force-pushed the mkdocs-tweak branch 4 times, most recently from 9d5b2de to 9949c91 Compare July 22, 2025 11:03
@aireilly aireilly marked this pull request as ready for review July 24, 2025 09:35
Copy link
Collaborator

@brian-dellabetta brian-dellabetta left a comment

Choose a reason for hiding this comment

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

looks good to me! thanks for setting this up

@aireilly aireilly changed the title [Docs] Clean up docs build [Docs] INFERENG-1089 - Clean up front page and add a Makefile Jul 30, 2025
@aireilly aireilly force-pushed the mkdocs-tweak branch 5 times, most recently from af54c30 to 856b86e Compare July 30, 2025 13:22
Copy link
Collaborator

@brian-dellabetta brian-dellabetta left a comment

Choose a reason for hiding this comment

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

gemini comment looks valid to me, otherwise LGTM

kylesayrs
kylesayrs previously approved these changes Jul 30, 2025
Copy link
Collaborator

@kylesayrs kylesayrs left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Collaborator

@brian-dellabetta brian-dellabetta left a comment

Choose a reason for hiding this comment

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

Thank you @aireilly !

@brian-dellabetta brian-dellabetta added the ready When a PR is ready for review label Jul 31, 2025
@aireilly aireilly dismissed stale reviews from brian-dellabetta and kylesayrs via 17ce3ac July 31, 2025 12:06
@aireilly aireilly force-pushed the mkdocs-tweak branch 2 times, most recently from 17ce3ac to 94ef00d Compare July 31, 2025 12:07
@aireilly aireilly enabled auto-merge (squash) July 31, 2025 12:08
@aireilly aireilly requested a review from kylesayrs August 5, 2025 14:06
Signed-off-by: Aidan Reilly <[email protected]>
@dsikka dsikka disabled auto-merge August 6, 2025 13:43
@dsikka dsikka merged commit 377a83e into vllm-project:main Aug 6, 2025
9 checks passed
dbarbuzzi pushed a commit that referenced this pull request Aug 8, 2025
SUMMARY:
* Adds a simple docs Makefile
* Cleans up the docs front page

Signed-off-by: Aidan Reilly <[email protected]>
Signed-off-by: Domenic Barbuzzi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready When a PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants