-
Notifications
You must be signed in to change notification settings - Fork 471
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
Rework internals #654
Rework internals #654
Conversation
6179fac
to
8df98fe
Compare
047d247
to
4b574fc
Compare
@zb140 this isn't ready for running on Windows yet, but here's a very quick overview of what this PR hopes to achieve. The primary goal is to enable new functionality that would be very tricky to implement with chezmoi's internal architecture in its current state. Specifically, these are:
chezmoi's internal architecture at the moment isn't terrible, but it does carry a fair amount of technical debt from its evolution over the past 18 months. Now chezmoi is stabilizing it's time for a clean-up. Internally, the clean-up does a number of things:
At the moment, I still have a number of unanswered questions that are not reflected in the code yet, so I wouldn't recommend diving in here unless you're particularly keen or bored :) |
4e6d491
to
cb64d07
Compare
This sounds super exciting! Let me know how I can help. |
2a752f1
to
1b9b564
Compare
6643661
to
aba7d03
Compare
2827de0
to
4ea4fea
Compare
ccfc595
to
b0a15cd
Compare
14c1ffa
to
286d53c
Compare
🎉 This is amazing! |
Thanks @zb140 - I'll open an issue explaining how to beta test in the next hour or so. |
This is a work-in-progress PR to update the internals of chezmoi to facilitate features like #423 and #424.