Skip to content

feat: consolidate all new labeling rules with comprehensive tests#46

Merged
ddjain merged 24 commits intomainfrom
feat/all-rules-consolidated
Oct 30, 2025
Merged

feat: consolidate all new labeling rules with comprehensive tests#46
ddjain merged 24 commits intomainfrom
feat/all-rules-consolidated

Conversation

@ddjain
Copy link
Member

@ddjain ddjain commented Oct 30, 2025

Summary

This PR consolidates all new auto-labeling rules with comprehensive test coverage (80%+).

New Rules Added

Database Rules

Test Rules (Issues #7, #8, #9)

  • test-only-change: Identifies test-only changes
  • test-missing: Flags source changes without corresponding tests
  • test-improvement: Detects test improvements/additions

Dependency Rules (Issues #10, #11, #12)

  • dependency-change: Detects dependency file modifications
  • new-dependency: Identifies new dependencies added
  • dependency-downgrade: Flags dependency version downgrades

CI/CD Rules

Infrastructure Rules

Semantics Rules (Issues #16, #17, #18, #19)

  • function-removed: Detects removed functions/classes
  • new-feature: Identifies new feature additions
  • non-functional-change: Flags documentation/non-code changes
  • refactor: Detects refactoring/rename operations

Security Rules

Meta Rules

Quality Metrics

✅ All 417 tests passing
✅ 80%+ branch coverage on all rules
✅ Comprehensive edge case handling
✅ Debug mode support for all rules

Issues Closed

Closes #6, #7, #8, #9, #10, #11, #12, #13, #15, #16, #17, #18, #19, #20, #21, #22

ddjain added 24 commits October 30, 2025 10:37
This PR consolidates all new auto-labeling rules into a single branch:

Database Rules:
- safe-migration (Issue #6): Safe migration detection

Test Rules (Issues #7, #8, #9):
- test-only-change: Test-only changes detection
- test-missing: Missing tests detection
- test-improvement: Test improvements detection

Dependency Rules (Issues #10, #11, #12):
- dependency-change: Dependency file changes
- new-dependency: New dependencies added
- dependency-downgrade: Dependency downgrades

CI/CD Rules:
- ci-change (Issue #13): CI workflow changes

Infrastructure Rules:
- infra-change (Issue #15): Infrastructure changes

Semantics Rules (Issues #16, #17, #18, #19):
- function-removed: Removed functions/classes
- new-feature: New feature detection
- non-functional-change: Non-code changes
- refactor: Refactoring detection

Security Rules:
- security-change (Issue #20): Security-sensitive area changes
- risky-code (Issue #21): Risky code patterns

Meta Rules:
- large-pr (Issue #22): Large PR detection
- Enhanced existing meta rules (missing-description, no-linked-issue, work-in-progress)

All rules include:
- Comprehensive unit tests with 80%+ branch coverage
- Proper metadata and descriptions
- Debug mode support
- Edge case handling
@ddjain ddjain merged commit 0eab14c into main Oct 30, 2025
5 checks passed
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.

Rule: Safe migration detection (safe-migration)

1 participant