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

Feature/laravel 9 support #81

Merged
merged 3 commits into from Feb 5, 2022

Conversation

bobbypiper
Copy link
Contributor

No description provided.

@antonkomarev
Copy link
Member

Thank you for the contribution @Zeoknight! Will look on it to merge ASAP.

@bobbypiper
Copy link
Contributor Author

@antonkomarev no problem, just updated PR as didn't realise the legacy-factories was included as part of travis process.

How would you go about testing locally? I've had to add legacy-factories package, run tests to ensure passing and then remove it again... any better solution?

@antonkomarev
Copy link
Member

Nope, just testing old versions in travis

@antonkomarev antonkomarev merged commit 65b6ec5 into cybercog:master Feb 5, 2022
@antonkomarev
Copy link
Member

@Zeoknight merged, I'll add tests to GitHub Actions and make a new release

@antonkomarev
Copy link
Member

antonkomarev commented Feb 5, 2022

Adding tests took more time than expected:

@antonkomarev
Copy link
Member

antonkomarev commented Feb 5, 2022

Laravel Ban v4.7 has been released! 🎉

@Zeoknight added to the Contributors list!

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