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

Unit Tests - PHP #737

Merged

Conversation

mvarendorff
Copy link
Contributor

@mvarendorff mvarendorff commented Oct 31, 2023

What does this PR do?

This PR adds generated unit tests to the PHP SDK.

Test Plan

Generate the SDK, install composer dependencies, then run vendor/bin/phpunit tests in the SDK's directory.

Related PRs and Issues

#680

Have you read the Contributing Guidelines on issues?

Yup


Discord username for swag as requested by Tessa: yestheory

@mvarendorff mvarendorff marked this pull request as ready for review October 31, 2023 10:26
@mvarendorff mvarendorff marked this pull request as draft October 31, 2023 10:28
@mvarendorff mvarendorff marked this pull request as ready for review October 31, 2023 10:33
@stnguyen90 stnguyen90 self-requested a review November 1, 2023 01:03
@mvarendorff mvarendorff force-pushed the feat-680-add-unit-tests-php branch from 3f84287 to 1113efa Compare November 4, 2023 18:37
@mvarendorff mvarendorff force-pushed the feat-680-add-unit-tests-php branch from 1113efa to b10525c Compare November 19, 2023 15:39
@lohanidamodar lohanidamodar merged commit cdcfb4b into appwrite:master Jan 2, 2024
35 checks passed
@mvarendorff mvarendorff deleted the feat-680-add-unit-tests-php branch January 2, 2024 08:56
@mvarendorff mvarendorff mentioned this pull request Feb 25, 2024
@gewenyu99
Copy link

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

@mvarendorff
Copy link
Contributor Author

Hey @gewenyu99 , thanks for the heads-up! My Discord username is @yestheory :) Is there a time-limit to responding? I am currently taking a bit of a break from Discord but I can hop on that account if you need me to!

This answer also applies to #735 and #736

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants