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

Enhancement: Implicitly enable no_unused_imports fixer #477

Merged
merged 2 commits into from
Dec 31, 2017

Conversation

localheinz
Copy link
Contributor

@localheinz localheinz commented Dec 30, 2017

This PR

  • implicitly enables the no_unused_imports fixer
  • runs composer style-fix

@localheinz localheinz changed the title Fix: Remove unused imports Enhancement: Enable no_unused_imports fixer Dec 30, 2017
@localheinz localheinz mentioned this pull request Dec 30, 2017
1 task
@localheinz localheinz changed the title Enhancement: Enable no_unused_imports fixer Enhancement: Implicitly enable no_unused_imports fixer Dec 30, 2017
@@ -15,7 +15,6 @@ $config
'array_syntax' => array('syntax' => 'long'),
'binary_operator_spaces' => false,
'concat_space' => array('spacing' => 'one'),
'no_unused_imports' => false,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since no_unused_imports is enabled as part of the @Symfony rule set, we implicitly enable it by removing this rule here.

@bighappyface bighappyface merged commit f6255c0 into jsonrainbow:master Dec 31, 2017
@localheinz localheinz deleted the fix/unused-import branch December 31, 2017 08:31
@localheinz
Copy link
Contributor Author

Thank you, @bighappyface and @erayd!

erayd pushed a commit to erayd/json-schema that referenced this pull request Feb 14, 2018
Enhancement: Implicitly enable no_unused_imports fixer
@erayd erayd mentioned this pull request Feb 14, 2018
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.

3 participants