-
Notifications
You must be signed in to change notification settings - Fork 169
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
Config autodiscovery #601
Comments
Dear @nlemoine , thanks a lot for this suggestions. The reasoning about the Not only I'll take them into consideration, but my long term goal, since I've introduced the support to wp-cli, is to move a lot of actions locally done by wordmove to
I've waited a long time before introducing wpcli integration; I've waited even more to make it the default. The goal was to wait for the dev community to adopt wp-cli as a standard WP development tool. I think and I hope the time has come. Unfortunately, Wordmove's codebase is in a bad state from the maintainability point of view and I really don't feel myself comfortable introducing other big changes into what I consider a mess. Because of this fact, I've decided to think a big refactor in the design of this software and I've actually started to write it down. I think to have found a nice path (https://github.com/welaika/wordmove/pull/587/files) but I'm still far away from the goal. My hope is that once finished, the sotware will be atomic, modular and someway functional, thus easier to modify and to extend for me and for other contributors. But I cannot lie: I'm neither half way from the goal at the moment. Hoping to be working for granting Wordmove others 9 years of life, I've to ask for everybody's patience in the meantime :) |
Hi @pioneerskies, Sorry for the late late reply. I'm finally getting a bit of spare time. You're right, |
@nlemoine now that I'm at 91 commit on that branch, I'm quite sure to be near the goal of simplifying the whole architecture. I've the hope this will make it easier to contribute; if not directly with PRs, with a chance to produce more technical insights. Morover implementing new features and fixing bug should become really easier. Unfortunately I 80% hate PHP but 95% love Ruby. Maybe when I'll ship a cleaner code, someone will be in a better place to port the whole thing in PHP...but I really couldn't stand to do any free-time/open-source work using PHP: it'd be a nightmare for me! 😅 |
Sorry, I had to ask 😄 |
I'm going to close this one for housekeepings. Discussion could eventually continue in PR #624 |
Hi @pioneerskies
Today, tooling is everywhere. This is great but configuring is often tedious and error prone. I often have to repeat some informations multiple times, at multiple locations. An so on when some value changes.
wp-cli
FTWThe whole discussion about adapting wordmove to the WordPress in a subdirectory structure (#597, #590, #591) and the possibilities offered by
wp-cli
gave me some ideas for improvements.Most, maybe all, config values could be handled automatically by wordmove.
wp config get
can grab all the necessary informations to make it work and yourmovefile.yaml
file could finally look like:Super ideally, it would even grab the different stages from
wp-cli.yml
: https://make.wordpress.org/cli/handbook/running-commands-remotely/So i could run
wordmove pull -d -e @production
.What do you think?
The text was updated successfully, but these errors were encountered: