-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Switch to using stubs for the correct version of WordPress being tested #16
Conversation
run: | | ||
if [ "${{ inputs.wp }}" == "latest" ]; then | ||
composer install --dev --prefer-dist | ||
elif [ "${{ inputs.wp }}" == "nightly" ]; then | ||
composer require --dev --update-with-dependencies --prefer-dist roots/wordpress-full="dev-main" | ||
else | ||
composer require --dev --update-with-dependencies --prefer-dist roots/wordpress-full="~${{ inputs.wp }}.0" php-stubs/wordpress-stubs="~${{ inputs.wp }}.0" | ||
fi |
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.
run: | | |
if [ "${{ inputs.wp }}" == "latest" ]; then | |
composer install --dev --prefer-dist | |
elif [ "${{ inputs.wp }}" == "nightly" ]; then | |
composer require --dev --update-with-dependencies --prefer-dist roots/wordpress-full="dev-main" | |
else | |
composer require --dev --update-with-dependencies --prefer-dist roots/wordpress-full="~${{ inputs.wp }}.0" php-stubs/wordpress-stubs="~${{ inputs.wp }}.0" | |
fi | |
if: "${{ inputs.wp == 'latest' }}" | |
run: "composer update --prefer-dist" | |
- name: "Install nightly dependencies" | |
if: "${{ inputs.wp == 'nightly' }}" | |
run: | | |
composer update --dev --with-all-dependencies --prefer-dist roots/wordpress-full:"dev-main" | |
- name: "Install dependencies" | |
if: "${{ inputs.wp != 'nightly' && inputs.wp != 'latest' }}" | |
run: | | |
composer update --dev --with-all-dependencies --prefer-dist roots/wordpress-full:"~${{ inputs.wp }}.0" php-stubs/wordpress-stubs:"~${{ inputs.wp }}.0" |
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.
Your repos don't have lock files. So install
will display a warning.
0e46355
to
85b8cd6
Compare
I'd love to do this but there's too many issues with dependency conflicts and unreliability of the older stubs. Closing this off, for now at least. |
No description provided.