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

Global refactoring adding type-hints, allow sf6 and php >=8 #72

Merged
merged 3 commits into from
Apr 14, 2022

Conversation

sadikoff
Copy link
Contributor

@sadikoff sadikoff commented Apr 13, 2022

Hello,

I'd like to add a lot of changes to the lib. Unfortunately, there are so many changes so it will be a totally BC break. But let's move forward.

  • Added type-hints
  • Bump version of Symfony to 5.4/6.0
  • Moved from Travis CI to GH action
  • Updated directory structure
  • Updated pusher/pusher-php-server to the latest

Cheers!

closes #71

@sadikoff sadikoff changed the title [WIP] Global refactoring adding type-hints, allow sf6 and php >=8 Global refactoring adding type-hints, allow sf6 and php >=8 Apr 14, 2022
@sadikoff
Copy link
Contributor Author

Hey @laupiFrpar

I hope you will have some time to review it =)

Cheers and have a nice day!

Copy link
Owner

@laupiFrpar laupiFrpar left a comment

Choose a reason for hiding this comment

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

Good job thanks

@laupiFrpar
Copy link
Owner

Strange. I canot see some tests results !

@laupiFrpar
Copy link
Owner

Hello,

I am at hospital. And I don't have a computer.

I use GitHub app and it is not simple to work with.

As soon as I get home, I'll run the tests locally and merge as soon as it's ok.

I get home next Tuesday.

Best regards,

@sadikoff
Copy link
Contributor Author

woh man... Hope nothing is too serious and you will get better soon!

Yeah tests are something complex, when you are moving to GH actions, they run only if master has a workflow file, and no tests are executed if workflow was just added in PR.

Probably there is something that needs to be adjusted so no rush.

just FYI
Screenshot 2022-04-14 at 11 24 51

@laupiFrpar
Copy link
Owner

Thank you

I merge and I update the version to 6.0.0.

Thank you again

@laupiFrpar laupiFrpar merged commit 1810780 into laupiFrpar:master Apr 14, 2022
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.

Is Symfony 6 supported?
2 participants