Skip to content

chore: Update version to 6.2.0 in configuration and installation scripts#27

Closed
Rudasingwa-kevin wants to merge 2 commits into
tang-vu:mainfrom
Rudasingwa-kevin:main
Closed

chore: Update version to 6.2.0 in configuration and installation scripts#27
Rudasingwa-kevin wants to merge 2 commits into
tang-vu:mainfrom
Rudasingwa-kevin:main

Conversation

@Rudasingwa-kevin
Copy link
Copy Markdown

Description

Type

  • feat – New feature
  • fix – Bug fix
  • refactor – Code restructuring
  • docs – Documentation
  • test – Tests
  • perf – Performance
  • chore – Maintenance

Changes

Architecture Area

  • Middleware chain (core/middleware.py)
  • Analysis / Skills (analysis/analyzer.py, analysis/skills.py)
  • Sub-agents (agents/registry.py)
  • Tools (tools/protocol.py)
  • Memory / Learning (orchestrator/memory.py)
  • PR Patrol (pr/patrol.py)
  • Pipeline (orchestrator/pipeline.py)
  • Other:

Pre-submit Checklist

⚠️ CI will auto-run on your PR. Please verify locally before pushing.

  • ruff check contribai/ passes (lint)
  • ruff format --check contribai/ tests/ passes (format)
  • pytest tests/ -v passes (356+ tests)
  • Code follows project conventions (async, type hints, Google docstrings)
  • from __future__ import annotations at top of new files
  • No hardcoded secrets
  • DCO signoff on all commits (git commit -s)

Related Issues

Copy link
Copy Markdown
Owner

@tang-vu tang-vu left a comment

Choose a reason for hiding this comment

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

LGTM. Real fixes:

  • json_parser.rs:342 — fixes E0308 compile error blocking cargo test
  • Cargo.toml + install scripts — sync version with README badge (v6.2.0)
  • patrol.rs:905 — silence unused-var warning

Thanks @Rudasingwa-kevin for catching the compile error.

@tang-vu tang-vu closed this in #30 Apr 27, 2026
tang-vu added a commit that referenced this pull request Apr 27, 2026
…rsion sync (#30)

Bundles credit from #27 (Co-authored-by: Rudasingwa-kevin) plus additional fixes for clippy 1.95 lints and pre-existing test failures on main.
@tang-vu
Copy link
Copy Markdown
Owner

tang-vu commented Apr 27, 2026

Đã được merge (gián tiếp) qua #30 với credit Co-authored-by: @Rudasingwa-kevin.

Mình bundle các fixes của PR này cùng với fixes cho clippy 1.95 lints và 4 failing tests pre-existing trên main vào một commit duy nhất để giữ lịch sử git sạch.

Cảm ơn bạn đã catch compile error ở json_parser.rs:342 — đó là lỗi nghiêm trọng đang block CI cho mọi PR khác. 🙏

Squash commit: cfe4568

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.

2 participants