-
Notifications
You must be signed in to change notification settings - Fork 11
Add modernization planning for developer utility #108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: mohlsen <3265497+mohlsen@users.noreply.github.com>
Co-authored-by: mohlsen <3265497+mohlsen@users.noreply.github.com>
|
@copilot i think the planning went a bit to far. This is a utility that developers can manually run, or include in scrips or workflows to validate their environment.
|
Co-authored-by: mohlsen <3265497+mohlsen@users.noreply.github.com>
You're absolutely right - I misunderstood the scope. Fixed in commit 4303f38. Changes made:
New focus:
The plan now emphasizes adding validators for container/cloud tools (like docker, kubectl, helm, aws CLI) to help developers validate their environment is ready for container/cloud development - not shipping check-engine itself as a container. |
|
@copilot i like the plan but not the order. lets do all the modeernization, improvements for dev productivity, tooling, testing, typescript conversion, api improvements, etc BEFORE we add new validations and enhancements features. I would like to have a good foundation in place, get it all working for existing validators, then start thinking about what enhancements to make. please reevaluate and update the plane |
Co-authored-by: mohlsen <3265497+mohlsen@users.noreply.github.com>
Updated in commit 568846a. The plan now follows a foundation-first approach: Phases 1-4 (Weeks 1-8): Complete ALL modernization first
Phases 5-6 (Weeks 9+): Add features after foundation is solid
This ensures existing validators are working great with modern tooling and tests before adding any new ones. |
Repository analysis requested for modernizing check-engine as a developer utility for validating local development environments.
Planning Document Created
MODERNIZATION_PLAN.md (~14KB) - Focused modernization roadmap for check-engine as a CLI utility with foundation-first phasing
What check-engine Is
What check-engine Is NOT
Proper Scope
The plan focuses on:
Phasing Strategy
Foundation First, Features Second - All modernization, dev productivity improvements, tooling, testing, TypeScript conversion, and API improvements happen BEFORE adding new validators and feature enhancements.
Phases 1-4: Foundation & Modernization (Weeks 1-8)
Phases 5-6: Features & Extensibility (Weeks 9+)
This approach ensures the codebase is stable and well-tested before adding new features, making refactoring easier and minimizing risk.
Current State
Note
Planning only—no code changes. Repository remains in working state. Implementation begins after review and approval.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.