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

Update Phive configuration #481

Merged

Conversation

jrfnl
Copy link
Contributor

@jrfnl jrfnl commented Aug 1, 2021


name: ⚙ Improvement
about: Update Phive configuration
labels: enhancement


Description

Phive has released version 0.15.0 which ups the minimum supported PHP version to PHP 7.3.

This updates the switch statement for Phive to take this into account.

Ref: https://github.com/phar-io/phive/releases

In case this PR introduced TypeScript/JavaScript code changes:

  • I have written test cases for the changes in this pull request
  • I have run npm run format before the commit.
  • I have run npm run lint before the commit.
  • I have run npm run release before the commit.
  • npm test returns with no unit test errors and all code covered.

In case this PR edits any scripts:

  • I have checked the edited scripts for syntax.
  • I have tested the changes in an integration test (If yes, provide workflow YAML and link).

P.S.: Hope I've done it right as I'm not familiar with writing code for GH actions.

@jrfnl jrfnl force-pushed the feature/update-phive-config branch from 9c4627b to 1fdc66c Compare August 1, 2021 17:38
@shivammathur shivammathur changed the base branch from develop to master August 1, 2021 17:48
@jrfnl

This comment has been minimized.

@shivammathur shivammathur changed the base branch from master to develop August 1, 2021 17:52
Phive has released version `0.15.0` which ups the minimum supported PHP version to PHP 7.3.

This adds the switch statement for Phive to take this into account.

Ref: https://github.com/phar-io/phive/releases
@jrfnl jrfnl force-pushed the feature/update-phive-config branch from 1fdc66c to f31d416 Compare August 1, 2021 17:53
@shivammathur shivammathur merged commit cd660de into shivammathur:develop Aug 1, 2021
@jrfnl jrfnl deleted the feature/update-phive-config branch August 1, 2021 17:54
@shivammathur
Copy link
Owner

I have merged this, the develop branch had some test commits which I will clean up later, sorry for the confusion.

@jrfnl
Copy link
Contributor Author

jrfnl commented Aug 1, 2021

@shivammathur No worries. Glad I was finally able to contribute something back.

shivammathur added a commit that referenced this pull request Aug 1, 2021
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