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

[Build] ignore string inflector phpstan #13138

Merged
merged 1 commit into from Sep 24, 2021

Conversation

SirDomin
Copy link
Contributor

Q A
Branch? master
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

@SirDomin SirDomin requested a review from a team as a code owner September 23, 2021 11:05
@GSadee GSadee added the Maintenance CI configurations, READMEs, releases, etc. label Sep 24, 2021
@GSadee GSadee merged commit 4a991f2 into Sylius:master Sep 24, 2021
@GSadee
Copy link
Member

GSadee commented Sep 24, 2021

Thank you, @SirDomin! 🎉

Zales0123 added a commit that referenced this pull request Sep 30, 2021
…or (Zales0123)

This PR was merged into the 1.10 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.10
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | the same as #13138, but for 1.10
| License         | MIT

It started to fail on 1.10 as well :/ I'm temporarily disabling the analysis and we need to figure out what is wrong with it 🖖 🔥 

Commits
-------

324e7c3 Ignore PHPStan analysis for StingInflector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance CI configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants