Skip to content
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

Fix Actions and update PHP matrix #174

Merged
merged 7 commits into from Jun 19, 2023
Merged

Conversation

whikloj
Copy link
Collaborator

@whikloj whikloj commented Jun 16, 2023

This includes the commits in #166, but also adds some tweaks based on phpstan runs and also adds PHP 8.2 and removes 7.3 from the build matrix.

I have left the

on:
  push:
    branches:
      - '**'

for now, I can remove it when you are ready. I can also re-add PHP 7.3 if you want to test it.

See here while it lasts for a test run.

Or you can ignore this and merge @danesteve's #166

@whikloj
Copy link
Collaborator Author

whikloj commented Jun 16, 2023

Oops I forgot that I moved the docker tags to be repository variables 🤦

ACTIVEMQ_IMAGE_TAG = 5.14.5-alpine

APOLLO_IMAGE_TAG = 1.7.1

ARTEMIS_IMAGE_TAG = 2.13.0-alpine

RABBITMQ_IMAGE_TAG = 3.6.11

@whikloj whikloj merged commit cf1fbd7 into stomp-php:master Jun 19, 2023
5 checks passed
@whikloj
Copy link
Collaborator Author

whikloj commented Jun 19, 2023

Rebasing so @danesteve's commits don't get collapsed into mine.

@whikloj whikloj mentioned this pull request Jun 19, 2023
@whikloj whikloj deleted the php-8.1-compat branch June 22, 2023 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants