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

PHP8.1 Update #5

Merged
merged 16 commits into from
Jun 10, 2023
Merged

PHP8.1 Update #5

merged 16 commits into from
Jun 10, 2023

Conversation

dolejska-daniel
Copy link
Owner

@dolejska-daniel dolejska-daniel commented Mar 20, 2022

  • refactor code to 8.1
  • update dependencies

@dolejska-daniel dolejska-daniel mentioned this pull request Mar 20, 2022
@dolejska-daniel dolejska-daniel self-assigned this Mar 20, 2022
@TheMY3
Copy link

TheMY3 commented Jun 9, 2023

@dolejska-daniel maybe do you need help with it? Or all ready for release?

@dolejska-daniel
Copy link
Owner Author

Hi @TheMY3! Most definitely! I would certainly appreciate any help. I'm honestly not quite sure what kind of state this PR is in and whether there is more work to be done, but I don't think that I considered it finished at the time.

@TheMY3
Copy link

TheMY3 commented Jun 9, 2023

@dolejska-daniel check one more files - looks like it is done 🤔 maybe just run travis builds? I can not see it here.
Or I can test this PR locally and then write here. But I use only some endpoints.

Repository owner deleted a comment from codecov-commenter Jun 10, 2023
@codecov
Copy link

codecov bot commented Jun 10, 2023

Codecov Report

❗ No coverage uploaded for pull request base (master@5e131ea). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   68.20%           
  Complexity        ?      275           
=========================================
  Files             ?       12           
  Lines             ?      629           
  Branches          ?        0           
=========================================
  Hits              ?      429           
  Misses            ?      200           
  Partials          ?        0           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dolejska-daniel dolejska-daniel marked this pull request as ready for review June 10, 2023 09:20
@dolejska-daniel dolejska-daniel merged commit 61d8306 into master Jun 10, 2023
4 checks passed
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.

None yet

2 participants