Skip to content

Conversation

@CarlSchwan
Copy link
Member

There are still a few issues reported by psalm but this is mainly some
usage of the private OC api and Sabre api. There is no way around that
for now.

This also port from ILogger to LoggerInterface

@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Apr 7, 2022
@CarlSchwan CarlSchwan self-assigned this Apr 7, 2022
@CarlSchwan CarlSchwan force-pushed the psalm-logger branch 4 times, most recently from edf0efe to 7af1704 Compare April 7, 2022 12:36
@come-nc
Copy link
Contributor

come-nc commented Apr 7, 2022

If applicable try to use nextcloud/.github#53

@come-nc
Copy link
Contributor

come-nc commented Apr 7, 2022

phpunit.xml needs to go in tests/

There is still a few issues reported by psalm but this is mostly some
usage of the private OC api and Sabre api. There is no way around that
for now.

This also port from ILogger to LoggerInterface

Signed-off-by: Carl Schwan <[email protected]>
@CarlSchwan CarlSchwan force-pushed the psalm-logger branch 2 times, most recently from 729adf0 to b7e2f4f Compare April 7, 2022 13:31
@CarlSchwan CarlSchwan force-pushed the psalm-logger branch 2 times, most recently from c4243de to 85c90ca Compare April 7, 2022 14:10
Signed-off-by: Carl Schwan <[email protected]>
There is no integration tests, so no need for that file

Signed-off-by: Carl Schwan <[email protected]>
@CarlSchwan CarlSchwan merged commit 7ea577a into master Apr 7, 2022
@delete-merged-branch delete-merged-branch bot deleted the psalm-logger branch April 7, 2022 15:30
@CarlSchwan
Copy link
Member Author

/backport to stable23

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Apr 7, 2022
@backportbot-nextcloud
Copy link

The backport to stable23 failed. Please do this backport manually.

@CarlSchwan
Copy link
Member Author

/backport to stable22

@backportbot-nextcloud
Copy link

The backport to stable22 failed. Please do this backport manually.

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

Labels

3. to review Waiting for reviews backport-request Pending backport by the backport-bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants