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

Migrate to Carbon2? #1233

Open
tdutrion opened this issue Nov 30, 2019 · 4 comments
Open

Migrate to Carbon2? #1233

tdutrion opened this issue Nov 30, 2019 · 4 comments

Comments

@tdutrion
Copy link
Contributor

Steps required to reproduce the problem

  1. clone the project
  2. run composer run setup-env

Expected Result

  • installation succeed

Actual Result

  • a line in the composer install states: Carbon 1 is deprecated, see how to migrate to Carbon 2.
Carbon 1 is deprecated, see how to migrate to Carbon 2.
https://carbon.nesbot.com/docs/#api-carbon-2
    You can run './vendor/bin/upgrade-carbon' to get help in updating carbon and other frameworks and libraries that depend on it.

Not sure how much work is involved, I have never used this solution.

@chartjes
Copy link
Contributor

@tdutrion Carbon 1 is a dependency of illuminate/support, which is in turn a dependency of wouterj/eloquent-bundle. Until such time the use of Eloquent is removed, this warning will be displayed. I will add notes to the README file that this is an expected warning message

@drbyte
Copy link

drbyte commented Nov 30, 2019

@tdutrion Carbon 1 is a dependency of illuminate/support, which is in turn a dependency of wouterj/eloquent-bundle. Until such time the use of Eloquent is removed, this warning will be displayed.

True, when still using illuminate/support versions less than 5.8.
It appears wouterj/eloquent-bundle is readying an update (1.1) to support newer versions: https://github.com/wouterj/WouterJEloquentBundle/releases

@chartjes chartjes reopened this Nov 30, 2019
@chartjes
Copy link
Contributor

chartjes commented Nov 30, 2019

@drbyte That is good to know! I have medium-term plans to get rid of the dependency on Eloquent, which is only there because I am using cartalyst/sentinel for auth. I believe the process that will have to be done is:

  • add Doctrine dependencies to project
  • create all the Doctrine entities needed
  • refactor existing code to use Doctrine instead of Eloquent
  • remove cartalyst/sentinel
  • add in symfony/security
  • refactor code to use Guard for auth
  • profit! (not sure about this step)

@drbyte
Copy link

drbyte commented Nov 30, 2019

get rid of the dependency on Eloquent, which is only there because I am using cartalyst/sentinel for auth

Sentinel was no doubt a suitable and ideal design choice at the time it was implemented.
But, as you mention, now having something more specific to the chosen framework is more ideal.
Sounds like the right direction to me.

  • profit! (not sure about this step)

That's always the hope, right!? Especially after doing all the work of the prior steps :P

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

No branches or pull requests

3 participants