/snapcraft

cli: init cmd migrated to docopts. #144

Merged
merged 1 commit into from Dec 3, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@sergiusens @elopio @kyrofa
@sergiusens
Collaborator

sergiusens commented Dec 3, 2015

Signed-off-by: Sergio Schvezov sergio.schvezov@canonical.com
@sergiusens
cli: init cmd migrated to docopts. 
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>

All checks have passed

2 successful checks
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+0.03%) to 78.263%
Details
5e201eb
@elopio
Member

elopio commented Dec 3, 2015

👍

sergiusens added a commit that referenced this pull request Dec 3, 2015

@sergiusens
Merge pull request #144 from sergiusens/05-docopts-init 
cli: init cmd migrated to docopts.
9408c16

@sergiusens sergiusens merged commit 9408c16 into snapcore:new-cli Dec 3, 2015
2 checks passed

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.03%) to 78.263%
Details

@sergiusens sergiusens deleted the sergiusens:05-docopts-init branch Dec 3, 2015

snapcraft/tests/test_commands_init.py
+ self.assertEqual(
+ 'snapcraft.yaml already exists!\n', fake_logger.output)
+
+ def test_init_must_write_snapcraft_yaml(self):
@kyrofa

kyrofa Dec 3, 2015

Member

Perhaps this test should check that the snapcraft.yaml is actually created instead of only checking the stdout?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment