chore: add issue templates, expand CONTRIBUTING.md, and project management setup#272
chore: add issue templates, expand CONTRIBUTING.md, and project management setup#272
Conversation
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
There was a problem hiding this comment.
Code Review
This pull request establishes a comprehensive project management infrastructure for the repository. It introduces standardized issue templates for bugs, features, performance, and datasets, alongside a significantly expanded CONTRIBUTING.md that details development setup, code style, and testing procedures. The PR also includes a design specification and a task-based implementation plan for migrating issues and configuring a project board. Review feedback focused on correcting pytest-asyncio marker usage, refining coverage commands, and improving the robustness of shell scripts for GitHub token retrieval.
…, and CONTRIBUTING.md Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
13-task plan covering labels, board, templates, CONTRIBUTING.md, issue migration, duplicate closure, PR linkages, and board automation setup. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Four YAML-based issue form templates: - 100-bug-report.yml: structured bug reporting - 200-feature-request.yml: feature proposals with motivation - 300-performance.yml: performance regressions with benchmark data - 400-dataset-integration.yml: new dataset/eval requests - config.yml: template chooser with Discussions link Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ue guidelines Replace minimal 10-line CONTRIBUTING.md with comprehensive guide covering: - Ways to contribute with links to issue templates - Development setup (venv, pip install, pre-commit, echo server) - Code style (ruff, mypy, conventional commits, serialization) - Testing (pytest markers, async mode, coverage, fixtures) - PR process and review expectations - Issue lifecycle and priority levels - MLCommons CLA requirements Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
One-way sync: when priority: or area: labels change on an issue, the corresponding board custom field is automatically updated. Labels are the single source of truth. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Remove .cursor/rules/ (migrated to CLAUDE.md/AGENTS.md) - Remove docs/superpowers/ plans and specs (local-only artifacts) - Add .cursor/ and docs/superpowers/ to .gitignore - Overhaul README.md: remove emojis, remove inline contributor list (use git log/ATTRIBUTION instead), align architecture section with AGENTS.md, add badges, streamline to match OSS best practices - Contributors section removed — credit lives in git history and ATTRIBUTION file Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Strict asyncio mode is configured globally in pyproject.toml via asyncio_mode = "strict". The marker does not accept a mode argument — passing it causes errors in recent pytest-asyncio versions. Fixed in: CONTRIBUTING.md, AGENTS.md, docs/DEVELOPMENT.md Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
CLA details are already in CONTRIBUTING.md — no need to duplicate in README. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Make it explicit that pre-commit must run before every commit, no exceptions. Hooks may modify files — stage changes and commit once. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Rashid Kaleem <230885705+arekay-nv@users.noreply.github.com>
2f2b15b to
fc391d7
Compare
| blank_issues_enabled: true | ||
| contact_links: | ||
| - name: Questions & Discussion | ||
| url: https://github.com/mlcommons/endpoints/discussions |
There was a problem hiding this comment.
Do we want to enable discussions?
Summary
What was done (via API, not in this PR)
The following was configured directly on GitHub via the API as part of this project management setup:
Test plan