-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add build step for wp-cli/wp-cli-shim
#592
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/phar
wp-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/phar
repository with acomposer.json
such as this one:And I think that's all that's needed.
To-do:
wp-cli/wp-cli-shim
or somethingFixes #134
Fixes wp-cli/ideas#145