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

Fossil VCS support for Composer #5467

Merged
merged 6 commits into from Jun 26, 2016

Conversation

Projects
None yet
4 participants
@bohwaz
Contributor

bohwaz commented Jun 24, 2016

This adds support for local and remote Fossil repositories as Composer packages.

Fossil is a popular DVCS that includes issues management, a wiki and more: https://www.fossil-scm.org/

@alcohol

This comment has been minimized.

Show comment
Hide comment
@alcohol

alcohol Jun 24, 2016

Member

Looks good, I think. Cannot say I ever heard of Fossil though. But then again, we still support SVN too.

Member

alcohol commented Jun 24, 2016

Looks good, I think. Cannot say I ever heard of Fossil though. But then again, we still support SVN too.

@alcohol alcohol added the Reviewed label Jun 24, 2016

@staabm

This comment has been minimized.

Show comment
Hide comment
@staabm

staabm Jun 24, 2016

Contributor

Misses at least some basic testcoverage

Contributor

staabm commented Jun 24, 2016

Misses at least some basic testcoverage

@alcohol alcohol added the Needs Test label Jun 24, 2016

@alcohol

This comment has been minimized.

Show comment
Hide comment
@alcohol

alcohol Jun 24, 2016

Member

Agreed.

Member

alcohol commented Jun 24, 2016

Agreed.

@alcohol alcohol added the Feature label Jun 24, 2016

@bohwaz

This comment has been minimized.

Show comment
Hide comment
@bohwaz

bohwaz Jun 25, 2016

Contributor

Will add some tests asap

Contributor

bohwaz commented Jun 25, 2016

Will add some tests asap

@bohwaz

This comment has been minimized.

Show comment
Hide comment
@bohwaz

bohwaz Jun 25, 2016

Contributor

Done.

Contributor

bohwaz commented Jun 25, 2016

Done.

@Seldaek

This comment has been minimized.

Show comment
Hide comment
@Seldaek

Seldaek Jun 26, 2016

Member

Thanks, looks like a fairly complete implementation.

Member

Seldaek commented Jun 26, 2016

Thanks, looks like a fairly complete implementation.

@Seldaek Seldaek added this to the 1.2 milestone Jun 26, 2016

@Seldaek Seldaek merged commit 664ba44 into composer:master Jun 26, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment