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

Consider config for NarrowReturnObjectTypeRule #13

Merged
merged 3 commits into from
Jun 17, 2024

Conversation

mitelg
Copy link
Contributor

@mitelg mitelg commented Jun 17, 2024

See #12

.gitignore Outdated Show resolved Hide resolved
@mitelg
Copy link
Contributor Author

mitelg commented Jun 17, 2024

as a run of the code style fixer fixed some unrelated files, I would assume, that https://github.com/rectorphp/type-perfect/blob/main/.github/workflows/code_analysis.yaml#L29 is not set up correctly 🤔 should this be run: composer check-cs --ansi instead?

@mitelg mitelg force-pushed the fix/consider-config branch from 7a24a2e to 06748a7 Compare June 17, 2024 12:35
@mitelg
Copy link
Contributor Author

mitelg commented Jun 17, 2024

I adjusted the job, because it should have failed for example on this run https://github.com/rectorphp/type-perfect/actions/runs/9547786462/job/26313537179

@samsonasik samsonasik merged commit 06a5fb5 into rectorphp:main Jun 17, 2024
6 checks passed
@samsonasik
Copy link
Member

Thank you @mitelg

@mitelg mitelg deleted the fix/consider-config branch June 17, 2024 13:03
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