Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

Applied fixes from FlintCI #590

Closed
wants to merge 1 commit into from
Closed

Conversation

soullivaneuh
Copy link
Member

No description provided.

@@ -60,8 +60,6 @@ class ProxyQuery implements ProxyQueryInterface

/**
* PHPCR ODM Document Manager.
*
* @var DocumentManager;
Copy link
Contributor

Choose a reason for hiding this comment

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

should this really be removed?

Copy link
Member

Choose a reason for hiding this comment

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

Maybe it’s spotted because it ends with a semicolon?

Copy link
Contributor

Choose a reason for hiding this comment

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

ok the semicolon sounds wrong. might be PHP-CS-Fixer/PHP-CS-Fixer#4913 - though here i think the fix would be to remove the semicolon

Copy link
Member

Choose a reason for hiding this comment

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

I fixed it in 2.x, lets close this one, so we get a new fresh one from afterwards...

@OskarStark OskarStark closed this Apr 14, 2020
@OskarStark OskarStark deleted the flintci-33312 branch April 14, 2020 11:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants