Skip to content

Detect supported PHP versions in range during CI instead of hardcoding them #52

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

Conversation

WyriHaximus
Copy link
Collaborator

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets blocks #50, relates to #51
License MIT

What's in this PR?

The proposed changes use a GitHub Actions I created to detect supported PHP version in range of composer.json.

Why?

To lower maintenance when new PHP releases come out in the future.

Checklist

  • Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

This was referenced May 9, 2022
Copy link
Contributor

@dbu dbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! i quite like this action, avoids messing up or forgetting versions.

…g them

The proposed changes use a GitHub Actions I created to detect supported PHP version in range of composer.json.
@WyriHaximus WyriHaximus force-pushed the detect-support-php-versions-in-range-during-ci-instead-of-hardcoding-them branch from 0ca34fe to c804b87 Compare June 24, 2022 17:28
@dbu
Copy link
Contributor

dbu commented Jun 27, 2022

the static analysis issue is not related to this PR. i will merge this, thanks!

@dbu dbu merged commit f7573f6 into php-http:master Jun 27, 2022
@WyriHaximus WyriHaximus deleted the detect-support-php-versions-in-range-during-ci-instead-of-hardcoding-them branch June 28, 2022 20:10
@WyriHaximus
Copy link
Collaborator Author

Awesome, I'll rebase #50 next

WyriHaximus added a commit to WyriHaximus-labs/react-adapter that referenced this pull request Jun 28, 2022
Move and also bump phpunit/phpunit to require dev as it was wrongfully place in require in php-http#52.
WyriHaximus added a commit to WyriHaximus-labs/react-adapter that referenced this pull request Jul 11, 2022
Move and also bump phpunit/phpunit to require dev as it was wrongfully place in require in php-http#52.
WyriHaximus added a commit to WyriHaximus-labs/react-adapter that referenced this pull request Jul 14, 2022
Move and also bump phpunit/phpunit to require dev as it was wrongfully place in require in php-http#52.
WyriHaximus added a commit to WyriHaximus-labs/react-adapter that referenced this pull request Jul 28, 2022
Move and also bump phpunit/phpunit to require dev as it was wrongfully place in require in php-http#52.
WyriHaximus added a commit to WyriHaximus-labs/react-adapter that referenced this pull request Jul 30, 2022
Move and also bump phpunit/phpunit to require dev as it was wrongfully place in require in php-http#52.
WyriHaximus added a commit to WyriHaximus-labs/react-adapter that referenced this pull request Jul 30, 2022
Move and also bump phpunit/phpunit to require dev as it was wrongfully place in require in php-http#52.
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.

2 participants