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

Merge Pomm-symfony-bridge with PommBundle ? #35

Open
stood opened this issue Jan 14, 2018 · 6 comments
Open

Merge Pomm-symfony-bridge with PommBundle ? #35

stood opened this issue Jan 14, 2018 · 6 comments

Comments

@stood
Copy link
Member

stood commented Jan 14, 2018

The bridge is using by Symfony and Silex, but Silex will be end of life the 18 june 2018

https://symfony.com/blog/the-end-of-silex

And the bridge have a big inconvenient, when we develop a functionality it's not easy to test impact on Symfony Bundle.

@stood
Copy link
Member Author

stood commented Jan 14, 2018

I'm sorry this issue is already open pomm-project/pomm-service-provider#7

@rdavaillaud
Copy link
Contributor

rdavaillaud commented Jun 5, 2018

@sanpii do you wait for any approval from the @chanmix51, or just waiting a PR?

If it's just a PR on the bundle, what will be the possible difficulties to expect?

@sanpii
Copy link
Member

sanpii commented Jun 6, 2018

@sanpii do you wait for any approval from the @chanmix51, or just waiting a PR?

I was waiting for the EOL. It’s done now 😃

If it's just a PR on the bundle, what will be the possible difficulties to expect?

Breaking changes will be difficult to avoid. I think (it’s just an idea before my morning tea) we could create src/PommBundle and src/SymfonyBridge.

@stood
Copy link
Member Author

stood commented Dec 8, 2019

After reflexion, I'm wondering if it's a good solution. In the futur, if we want develop a new Bundle with a new component Pomm (other ModelManager for example) we will to duplicate the code.

What do you think?

@rdavaillaud
Copy link
Contributor

🤔
All we are doing here, is merging 2 components which are non longer needed to be separated.

I think it's a good thing, we could then bring some new features to this bundle, maybe split it in PommFoundationBundle, PommModelBundle, PommWhateverBundle later.

@stood
Copy link
Member Author

stood commented Dec 9, 2019

Hmm, I think it's more complicated. The bridge was develop for to work with silex and symfony. But tomorrow if we want to develop a new Bundle PommWhateverBundle the DatabaseCollector and TypeExtractor for example will be common.

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