-
Notifications
You must be signed in to change notification settings - Fork 35
Add build step for wp-cli/wp-cli-shim
#592
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: main
Are you sure you want to change the base?
Conversation
|
@szepeviktor What do you think of this? Does it solve your needs? |
|
@swissspidy The repository was created: https://github.com/wp-cli/wp-cli-shim |
|
@schlessera Thanks! Can you add me to the repo so I can add an initial |
wp-cli/pharwp-cli/wp-cli-shim
|
@swissspidy Done. |
|
@schlessera OK so I pushed a branch intending to create a PR but because it was the first branch that became the default 🤦 https://github.com/wp-cli/wp-cli-shim Can you make |
|
@schlessera OK, so now I suppose what's left is making sure that the Once it does, we can tag what's currently in the repo with Not sure if we need |
|
@schlessera wanna give this a try for next release? |
| uses: ad-m/github-push-action@master | ||
| with: | ||
| github_token: ${{ secrets.ACTIONS_BOT }} | ||
| branch: gh-pages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| branch: gh-pages |

Assumes the existence of a new
wp-cli/pharrepository with acomposer.jsonsuch as this one:{ "name": "wp-cli/wp-cli-shim", "description": "Composer-based WP-CLI Phar", "license": ["MIT"], "require": { "php": ">=5.6" }, "conflict": { "wp-cli/wp-cli" : "*" }, "bin": ["wp-cli.phar"] }And I think that's all that's needed.
To-do:
wp-cli/wp-cli-shimor somethingFixes #134
Fixes wp-cli/ideas#145