Skip to content

Conversation

@sheldonvaughn
Copy link

No description provided.

@sheldonvaughn sheldonvaughn requested a review from a team as a code owner April 20, 2023 18:22
@sheldonvaughn sheldonvaughn requested review from jonatascastro12 and removed request for a team April 20, 2023 18:22
@sheldonvaughn sheldonvaughn changed the title Update WorkOS PHP to v3.2.0 and add Laravel 10 Update WorkOS PHP to v3.2.0 and add Laravel 10 Support Apr 20, 2023
@sheldonvaughn sheldonvaughn changed the title Update WorkOS PHP to v3.2.0 and add Laravel 10 Support Update WorkOS PHP to v3.2.0 and Add Laravel 10 Support Apr 20, 2023
@sheldonvaughn sheldonvaughn changed the title Update WorkOS PHP to v3.2.0 and Add Laravel 10 Support Update WorkOS PHP to v3.2.0 Apr 20, 2023
commands:
- bash bin/restore-cache-and-update-deps 7
- composer run-script test
- composer run-script test

Choose a reason for hiding this comment

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

Do we need these changes here? Looks like just whitespace.

composer.json Outdated
"test": "php vendor/bin/phpunit tests"
}
}
}

Choose a reason for hiding this comment

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

Could we keep the newline at the end of the file?

Copy link

@maxdeviant maxdeviant left a comment

Choose a reason for hiding this comment

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

A couple of minor comments, but otherwise LGTM

@sheldonvaughn sheldonvaughn merged commit ba3884d into main Apr 20, 2023
@sheldonvaughn sheldonvaughn deleted the updateWorkosPHPv3 branch April 20, 2023 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants