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

Restoring CheckModules as deprecated code #1939

Merged

Conversation

pedrofsn
Copy link
Contributor

@pedrofsn pedrofsn commented Aug 7, 2024

Restoring @arnaudgiuliani previous commit and adding fancy deprecation. And of course, unit test.
It will avoid an huge breaking change for big projects migrating Koin 3.x to Koin 4.x.

@pedrofsn
Copy link
Contributor Author

pedrofsn commented Aug 7, 2024

Fixes #1938

@arnaudgiuliani
Copy link
Member

Ok sounds fair to keep it back 👍

@arnaudgiuliani arnaudgiuliani added this to the 4.0-RC2 milestone Aug 12, 2024
@arnaudgiuliani
Copy link
Member

just updated main branch to let examples uses RC1 version instead of Beta1

@arnaudgiuliani
Copy link
Member

can you rebase on last branch?

@pedrofsn
Copy link
Contributor Author

I'm without a computer right now, so I did using GH PR flow. 🙃
I expect it worked as well.

@arnaudgiuliani
Copy link
Member

Ok, I'll check

@arnaudgiuliani arnaudgiuliani self-requested a review August 27, 2024 15:34
@arnaudgiuliani arnaudgiuliani merged commit b2452ad into InsertKoinIO:4.0.0 Aug 27, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants