autopkgtests: run the install examples tests in classic #481

Merged
merged 10 commits into from May 13, 2016

Conversation

Projects
None yet
3 participants
Member

elopio commented Apr 21, 2016

LP: #1572764

Member

elopio commented Apr 21, 2016

I'm internet handicapped, so I'm throwing this to CI. Plese wait for the tests to pass before doing a final review.

Member

kyrofa commented Apr 22, 2016

Looks good to me, though the autopkg tests are failing 👍

Member

elopio commented Apr 22, 2016

Let's better wait for my other fixes on the examples suite. Thanks for the review kyle.

Collaborator

sergiusens commented Apr 26, 2016

Eager to see you update the branch 😄

Member

elopio commented Apr 26, 2016

almost forgot about this. Pushed and fingers crossed.

Member

elopio commented Apr 27, 2016

retest this please

Member

elopio commented Apr 28, 2016

retest this please

Member

elopio commented Apr 28, 2016

retest this please

Collaborator

sergiusens commented Apr 29, 2016

retest this please

Member

elopio commented Apr 29, 2016

retest this please

Member

elopio commented Apr 30, 2016

retest this please

Leo Arias and others added some commits Apr 21, 2016

Collaborator

sergiusens commented May 3, 2016

\o/

Member

kyrofa commented May 3, 2016

As I mentioned in standup I'm not a huge fan of focusing our examples on our test cases rather than as examples for developer usage, but I understand the difficulty here and I'm 👍 .

Member

elopio commented May 3, 2016

@kyrofa, well, we can have both. I can make it that it exits only if it receives a specific message. Care to report a bug about that?

Member

kyrofa commented May 3, 2016

I can make it that it exits only if it receives a specific message. Care to report a bug about that?

Good idea-- done, with a low priority.

Member

elopio commented May 13, 2016

restest this please

@elopio elopio merged commit ca2c239 into snapcore:master May 13, 2016

4 checks passed

Examples tests Success
Details
autopkgtest Success
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 96.242%
Details
Member

elopio commented May 13, 2016

\o/\o/\o/\o/\o/\o/\o/\o/\o/\o/\o/

@elopio elopio deleted the elopio:bug/1572764/isntall-on-classic branch May 13, 2016

kalikiana pushed a commit to kalikiana/snapcraft that referenced this pull request Apr 6, 2017

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