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

[Scoper] Using different scoper.php for php 7.0 #6295

Merged
merged 5 commits into from
May 2, 2021

Conversation

samsonasik
Copy link
Member

revert original scoper.php

@samsonasik samsonasik enabled auto-merge (squash) May 2, 2021 18:08
@samsonasik samsonasik disabled auto-merge May 2, 2021 18:08
@samsonasik samsonasik enabled auto-merge (squash) May 2, 2021 18:11
@samsonasik
Copy link
Member Author

@TomasVotruba if this still make php 7.1 fail, then there is something wrong with the git add process itself, as previously, it still works on php 7.1. Let's try.

@samsonasik samsonasik merged commit e1c87b5 into main May 2, 2021
@samsonasik samsonasik deleted the using-different-scoper branch May 2, 2021 18:19
@@ -5,7 +5,7 @@ name: Build Scoped Rector PHP 7.0
on:
push:
branches:
- main
- never
Copy link
Member Author

Choose a reason for hiding this comment

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

temporary disable scoped 7.0

@samsonasik
Copy link
Member Author

the php 7.1 scoped seems working again.

@TomasVotruba
Copy link
Member

Thanks 👍

https://github.com/rectorphp/rector-prefixed is passing ✔️

TomasVotruba added a commit that referenced this pull request Sep 8, 2024
rectorphp/rector-src@1ebb7aa [CodeQuality] Allow transform static to self on final class on ConvertStaticPrivateConstantToSelfRector (#6295)
TomasVotruba added a commit that referenced this pull request Sep 8, 2024
rectorphp/rector-src@1ebb7aa [CodeQuality] Allow transform static to self on final class on ConvertStaticPrivateConstantToSelfRector (#6295)
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