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

Bump dependencies to fix AppVeyor failures #1302

Merged
merged 4 commits into from
Aug 20, 2020

Conversation

sanmai
Copy link
Member

@sanmai sanmai commented Aug 20, 2020

No description provided.

@sanmai

This comment has been minimized.

@maks-rafalko
Copy link
Member

Seems like after upgrading PHPParser, SA tools are not happy

@sanmai
Copy link
Member Author

sanmai commented Aug 20, 2020

These are fixed, but now we have a different kind:
https://travis-ci.com/github/infection/infection/jobs/375603434#L413

Which looks like this:
symfony/symfony#29341 (and fix symfony/symfony#29349)

@sanmai sanmai force-pushed the pr/2020-08/composer-bump-appveyor-fix branch from 889fab5 to 63ae3d1 Compare August 20, 2020 07:21
@sanmai
Copy link
Member Author

sanmai commented Aug 20, 2020

@maks-rafalko everything is green at last!

Copy link
Member

@maks-rafalko maks-rafalko left a comment

Choose a reason for hiding this comment

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

Cool!

@sanmai sanmai merged commit ab6b83e into infection:master Aug 20, 2020
sanmai added a commit to sanmai/infection that referenced this pull request Oct 2, 2020
* Bump dependencies to fix AppVeyor failures

* Force COLUMNS to be 100

* Fix autoreview issues

* Attempt to address this 'The process has been signaled with signal "11".'
sanmai added a commit that referenced this pull request Oct 2, 2020
* Bump dependencies to fix AppVeyor failures (backport of #1302)
* Limit PHP Parser version (backport from #1309)
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.

None yet

2 participants