Skip to content

Conversation

@szepeviktor
Copy link
Contributor

From e863dce

@szepeviktor szepeviktor marked this pull request as draft August 2, 2022 08:06
@szepeviktor szepeviktor marked this pull request as ready for review August 2, 2022 08:13
@szepeviktor
Copy link
Contributor Author

💡 There is no CS rule to spot unused imports: use DOMNode;

@theseer
Copy link
Member

theseer commented Aug 2, 2022

bulb There is no CS rule to spot unused imports: use DOMNode;

We probably should add that ;)

@szepeviktor
Copy link
Contributor Author

'no_unused_imports' => true,
?

@szepeviktor
Copy link
Contributor Author

I see! That workflow does not run on PR-s.

@theseer
Copy link
Member

theseer commented Aug 2, 2022

I see! That workflow does not run on PR-s.

Feel free to add that in a PR :) I'm rather busy today, sorry..

@szepeviktor
Copy link
Contributor Author

@theseer Friendly ping 🏓

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants