Skip to content

Conversation

@Arthur-Sk
Copy link
Contributor

No description provided.

@Arthur-Sk
Copy link
Contributor Author

Arthur-Sk commented Mar 21, 2024

Hey @belamov, can you take a look at this PR, please? We cannot update Laravel to the latest version without these changes

@belamov
Copy link
Owner

belamov commented Mar 21, 2024

hey @Arthur-Sk thanks for pr!

could you please update testing matrix aswell, so we can make sure this packege is working on laravel 11?

https://github.com/belamov/postgres-range/blob/master/.github/workflows/main.yml#L37

@belamov belamov self-assigned this Mar 21, 2024
@codecov
Copy link

codecov bot commented Mar 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.51%. Comparing base (6ca79c1) to head (a15b166).

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #26   +/-   ##
=========================================
  Coverage     99.51%   99.51%           
  Complexity       81       81           
=========================================
  Files            21       21           
  Lines           208      208           
=========================================
  Hits            207      207           
  Misses            1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@belamov
Copy link
Owner

belamov commented Mar 22, 2024

some tests are failng

please exclude them from matrix if this is dependency problem, for example: https://github.com/belamov/postgres-range/actions/runs/8387175011/job/22969055071?pr=26

@Arthur-Sk
Copy link
Contributor Author

Sure, I'm in progress already

@Arthur-Sk
Copy link
Contributor Author

Good to go

@belamov belamov merged commit 39f0beb into belamov:master Mar 22, 2024
@belamov
Copy link
Owner

belamov commented Mar 22, 2024

thanks a lot!

@belamov
Copy link
Owner

belamov commented Mar 22, 2024

@Arthur-Sk released new minor version, so you shold be able to update laravel iin your project

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