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

Update integrations tests #3817

Merged
merged 1 commit into from Jul 2, 2018
Merged

Update integrations tests #3817

merged 1 commit into from Jul 2, 2018

Conversation

SpacePossum
Copy link
Contributor

No description provided.

@@ -153,7 +153,6 @@ public function provideFixersPriorityCases()
array($fixers['phpdoc_summary'], $fixers['phpdoc_trim']),
array($fixers['phpdoc_to_comment'], $fixers['no_empty_comment']),
array($fixers['phpdoc_to_comment'], $fixers['phpdoc_no_useless_inheritdoc']),
array($fixers['phpdoc_var_without_name'], $fixers['phpdoc_trim']),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

there is no priority issue between these fixers

@SpacePossum SpacePossum added this to the 2.2.21 milestone Jun 6, 2018
@julienfalque
Copy link
Member

julienfalque commented Jun 6, 2018

Please rebase onto 2.12.

@julienfalque julienfalque modified the milestones: 2.2.21, 2.12.1 Jun 6, 2018
@SpacePossum
Copy link
Contributor Author

SpacePossum commented Jun 6, 2018

we're not doing a 2.2.21?

@julienfalque
Copy link
Member

Nope: https://medium.com/@keradus/to-make-a-difference-8dce5f7cf9bf

Though branch 2.12 doesn't exist yet, I guess master is fine for now.

@keradus keradus modified the milestones: 2.12.1, 2.12.2 Jun 8, 2018
@keradus keradus changed the base branch from 2.2 to 2.12 June 13, 2018 20:04
@SpacePossum SpacePossum added the RTM Ready To Merge label Jun 22, 2018
@SpacePossum
Copy link
Contributor Author

resolved conflicts after target branch was changed, forced pushed

@SpacePossum SpacePossum merged commit 73adc32 into PHP-CS-Fixer:2.12 Jul 2, 2018
SpacePossum added a commit that referenced this pull request Jul 2, 2018
This PR was merged into the 2.12 branch.

Discussion
----------

Update integrations tests

Commits
-------

73adc32 Update integrations tests
@SpacePossum SpacePossum removed the RTM Ready To Merge label Jul 2, 2018
@keradus keradus deleted the 2_2_integrations_tests branch July 2, 2018 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants