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

deps: Allow symfony 7 #40

Merged
merged 3 commits into from
Nov 15, 2024
Merged

deps: Allow symfony 7 #40

merged 3 commits into from
Nov 15, 2024

Conversation

aszenz
Copy link
Contributor

@aszenz aszenz commented Dec 9, 2023

Remove sensio it's deprecated

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Documented? no
Fixed tickets #39
License MIT

Allow symfony 7

Remove sensio it's deprecated
Copy link
Member

@mcg-web mcg-web left a comment

Choose a reason for hiding this comment

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

Thank you @aszenz! Can you fix this one please?

composer.json Outdated Show resolved Hide resolved
@Dipesh79
Copy link

@mcg-web can you please new PRs. I need this is symfony 7 project. Can you please review new PRs and merge it?

Co-authored-by: Jeremiah VALERIE <[email protected]>
@aszenz aszenz requested a review from mcg-web April 17, 2024 12:19
@aat-antoine
Copy link

aat-antoine commented May 16, 2024

Like @Dipesh79 said, @mcg-web can you review/merge the PR ?
Merci 👍

mcg-web
mcg-web previously approved these changes May 16, 2024
@enricobono
Copy link

Hello @mcg-web.
In our company we use this library and we'd like to help in making the lib compatible with Symfony 7.
This PR seems to solve the problem. Do you need help with the review? Can we help in any way? :)

@mcg-web mcg-web merged commit c43f73d into overblog:master Nov 15, 2024
@mcg-web
Copy link
Member

mcg-web commented Nov 15, 2024

Thank you everyone and sorry for the late reply ! I will release a new version next week.

@enricobono
Copy link

Thanks @mcg-web 🙌

I guess this will unlock thecodingmachine/graphqlite-bundle#206 too ;)

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.

6 participants