Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Zend Dom and standalone #5878

Closed
harikt opened this issue Feb 27, 2014 · 6 comments
Closed

Zend Dom and standalone #5878

harikt opened this issue Feb 27, 2014 · 6 comments

Comments

@harikt
Copy link
Contributor

harikt commented Feb 27, 2014

Hi ,

Considering the removal of stdlib #5865 and the package has become truly independent will you be considering to move the tests also to https://github.com/zendframework/Component_ZendDom , so people can send PR or can create issues at https://github.com/zendframework/Component_ZendDom .

I was looking at ZendXML and thought of this.

Thank you.

@Ocramius
Copy link
Member

@harikt that is not possible and would make it impossible to manage the component/releases together with the framework.

The component packages are generated from a subtree split and are to be considered read-only.

@harikt
Copy link
Contributor Author

harikt commented Feb 27, 2014

@Ocramius how does https://github.com/zendframework/ZendXml works ?

It is not a subtree split according to my knowledge ( Looking at the commits ) .

So I was asking whether the tests can also be moved like the https://github.com/zendframework/ZendXml .

@Ocramius
Copy link
Member

@harikt moving the component out of ZF2 and into a separate repo would require making ZF2 require zendframework/zendframework depending on zendframework/zend-dom for consistency (or break BC).

It can be done, but I'm not sure if it is worth it. It would only make things harder for maintainers that would have to hop back and forth through repositories.

I'd personally prefer to have the issue stay in the main repo to be able to track them in a meaningful way.

What is the big deal with moving them out? What are the main advantages?

EDIT: yes, ZendXml is out of the main repo itself, looks like a new project though.

@harikt
Copy link
Contributor Author

harikt commented Feb 27, 2014

@Ocramius I don't understand how it requires zendframework/zendframework . In zendframework/zendframework composer you can add he zendframework/zend-dom according to my knowledge.

Moving to a separate one means tests and PR need only done to the same repo. Just a suggestion as I see ZendXml.

@Ocramius
Copy link
Member

@harikt no, what I mean is that users upgrading and using the component would either need a new additional package or be forced to get (transitively) a new package.

I just don't see the benefits tbh - if you have a subtree split of the existing package that provides you with the reduced zipball etc etc, then why not just keep it as it is until 3.x if needed?

There must be stronger reasons to drive such a change in my opinion. It is just going to be more problematic with documentation, releases, consistency with the fw and most probably would just be forgotten in a corner.

@Maks3w
Copy link
Member

Maks3w commented Feb 27, 2014

Sorry guys. Issue tracker is not the best place for take this discussions. Better use the mailing list

@Maks3w Maks3w closed this as completed Feb 27, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants