Align repository with llm-d repo template#649
Open
InfraWhisperer wants to merge 1 commit intollm-d:mainfrom
Open
Align repository with llm-d repo template#649InfraWhisperer wants to merge 1 commit intollm-d:mainfrom
InfraWhisperer wants to merge 1 commit intollm-d:mainfrom
Conversation
Brings this repo into conformance with the llm-d Go repo template (https://github.com/llm-d/llm-d-go-template). All 15 items from llm-d#639 are addressed: Governance docs: CODE_OF_CONDUCT.md, CONTRIBUTING.md, PR_SIGNOFF.md, SECURITY.md — copied verbatim from the template. Linting/config: .gitattributes, .hadolint.yaml, .pre-commit-config.yaml, .yamllint.yml, pyproject.toml added from template. Renamed .typos.toml to _typos.toml to match template naming convention; merged repo-specific false-positive words (k8s, KServe, vLLM) and file exclusions with the template's baseline config. GitHub metadata: .github/CODEOWNERS (defaults to @llm-d/llm-d-inference-scheduler-maintainers), .github/PULL_REQUEST_TEMPLATE.md. Workflows: .github/workflows/copilot-setup-steps.yml for gh-aw extension. Issue templates: Converted bug_report.md and feature_request.md to YAML form format (bug_report.yml, feature_request.yml) with structured fields; removed the old markdown templates. Closes llm-d#639 Signed-off-by: Raghav Potluri <raghav.potluri21@gmail.com>
InfraWhisperer
force-pushed
the
align-repo-template
branch
from
188e072 to
5ffdf8c
Compare
15 tasks
elevran
added
the
do-not-merge/hold
Collaborator
|
this PR is on hold until the original issue is triaged and accepted |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Brings this repo into full conformance with the llm-d Go repo template. Addresses all 15 items from #639.
Why is this change needed?
The repo was missing standard governance docs, linting configs, and GitHub metadata that all llm-d repos should carry. Without these, contributors hit friction on DCO sign-off, code review expectations, and security reporting — and CI tooling (hadolint, yamllint, pre-commit) had no config to anchor on.
What changed
Governance docs added:
CODE_OF_CONDUCT.md— Contributor Covenant v1.4CONTRIBUTING.md— lazy consensus process, proposal requirements, code review gates, DCO sign-off, testing tiersPR_SIGNOFF.md— DCO sign-off vs signing, GPG/SSH key setup, commit repair instructionsSECURITY.md— vulnerability reporting via llm-d-security-reporting@googlegroups.comLinting / config added:
.gitattributes— linguist-generated markers for lock files.hadolint.yaml— Dockerfile lint config with standard ignore list.pre-commit-config.yaml— trailing whitespace, YAML/JSON checks, shellcheck, hadolint, markdownlint, yamllint.yamllint.yml— max 250 line length, document-start disabledpyproject.toml— Ruff config for any Python toolingTypos config renamed:
.typos.toml→_typos.tomlto match template naming. Merged repo-specific false-positive words (k8s,KServe,vLLM) and file exclusions with the template baseline (AKS,IST,ND).GitHub metadata added:
.github/CODEOWNERS— defaults to@llm-d/llm-d-inference-scheduler-maintainers.github/PULL_REQUEST_TEMPLATE.md— structured PR templateWorkflow added:
.github/workflows/copilot-setup-steps.yml— installs gh-aw extensionIssue templates converted:
bug_report.md→bug_report.yml(YAML form with structured fields: contact, description, version, repro steps, environment, logs)feature_request.md→feature_request.yml(YAML form with problem statement, proposed solution, alternatives, willingness to contribute)How was this tested?
.typos.toml,bug_report.md,feature_request.md) are removedCloses #639