Skip to content

fix: Fix PSR issue in database library#1997

Merged
christyjacob4 merged 1 commit intoappwrite:masterfrom
srijit2002:master
Oct 19, 2021
Merged

fix: Fix PSR issue in database library#1997
christyjacob4 merged 1 commit intoappwrite:masterfrom
srijit2002:master

Conversation

@srijit2002
Copy link
Copy Markdown
Contributor

What does this PR do?

Appwrite uses some custom libraries (and extensions of libraries) that are all stored in the src/Appwrite folder. Some of these files may not be fully compliant with the PSR coding standards, and my pr fix those type issues present in database library.

Related PRs and Issues

Fixes #1938

Have you read the Contributing Guidelines on issues?

Yes!

@christyjacob4
Copy link
Copy Markdown
Contributor

Waiting for tests to pass

@christyjacob4
Copy link
Copy Markdown
Contributor

Screenshot from 2021-10-19 10-34-35

The tests passed for one of the environments so I'm going to merge this one.

@christyjacob4 christyjacob4 added the hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later label Oct 19, 2021
@christyjacob4 christyjacob4 merged commit 88cfcb9 into appwrite:master Oct 19, 2021
@christyjacob4
Copy link
Copy Markdown
Contributor

THANK YOU so much! All changes merged 🥳
Please reach out to me on our Discord server if you would like to claim your Appwrite swag!

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

Labels

hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Help us fix PSR issues in the Database library [#master]

2 participants