Skip to content

[dbal 3.8] Add MigrateQueryBuilderResetQueryPartRector #455

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

Merged
merged 1 commit into from
Jul 6, 2025

Conversation

simonschaufi
Copy link
Contributor

Resolves: #453

@TomasVotruba
Copy link
Member

Looks very good, thank you 😇

@TomasVotruba TomasVotruba merged commit c883a96 into rectorphp:main Jul 6, 2025
6 checks passed
@simonschaufi
Copy link
Contributor Author

can you please set in the repo settings that HEAD branches are automatically removed once they are merged?

@simonschaufi simonschaufi deleted the issue-453 branch July 6, 2025 22:40
@TomasVotruba
Copy link
Member

Screenshot_20250707-070516

It's already enabled 👍

@simonschaufi
Copy link
Contributor Author

ah ok. I still had to delete my branch. Maybe that is only the case for branches of the same repo, not for forks.

@TomasVotruba
Copy link
Member

TomasVotruba commented Jul 7, 2025

Indeed, I think we can't delete branches in forks. That would be too invasive.

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.

Missing migration for resetQueryPart
2 participants