-
-
Notifications
You must be signed in to change notification settings - Fork 336
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
[PHP 8.1] Add nested attributes arguments + keep short ORM\* aliases #1278
Conversation
d979a42
to
b08fc59
Compare
6288688
to
a60624f
Compare
a60624f
to
480e260
Compare
bfa5afa
to
cb84955
Compare
- run: vendor/bin/paratest -p4 --runner=WrapperRunner ${{ matrix.path }} --colors | ||
- run: vendor/bin/paratest -p5 --runner=WrapperRunner ${{ matrix.path }} --colors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@samsonasik Just FYI, I don't use paratest at all, so I might be missing something.
This job was constantly failing on this PR, for error 137. AFAIK run out of memory.
I tried to change -p
to 5
and it seems working 👍 Let me know if there is a better way to handle it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TomasVotruba thank you, it was slower with 5
, but seems faster now 👍
Follow up to #1266
See rectorphp/rector#6792
Keep short ORM* like aliases
Closes rectorphp/rector#6659, closes rectorphp/rector#6565