/snapd

Rename /apps -> /snaps (same for data, and user-data) #308

Merged
merged 4 commits into from Jan 12, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@mvo5 @niemeyer
@mvo5
Collaborator

mvo5 commented Jan 8, 2016

Also remove /gadget which has some unfortuante side-effects
in the lightweight code. We need a better concept here as
lightweight does not deals with os/kernel types at all
currently.

Oh, forgot to mention this - this will break every installed apps.
@mvo5
Rename /apps -> /snaps (same for data, and user-data) 
Also remove /gadget which has some unfortuante side-effects
in the lightweight code. We need a better concept here as
lightweight does not deals with os/kernel types at all
currently.

Some checks were not successful

1 failing and 2 successful checks
@snappy-m-o
Integration tests — 20 tests run, 0 skipped, 10 failed.
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage decreased (-0.01%) to 67.619%
Details
4c47115
@niemeyer
Contributor

niemeyer commented Jan 8, 2016

LGTM!
@mvo5
Ensure /snaps is created at package build time

Some checks were not successful

1 failing and 2 successful checks
@snappy-m-o
Integration tests — 20 tests run, 0 skipped, 7 failed.
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage decreased (-0.03%) to 67.595%
Details
883a532
@mvo5
Collaborator

mvo5 commented Jan 9, 2016

retest this please
@mvo5
Collaborator

mvo5 commented Jan 9, 2016

retest this please
@mvo5
Collaborator

mvo5 commented Jan 9, 2016

retest this please
@mvo5
Collaborator

mvo5 commented Jan 10, 2016

retest this please
@mvo5
Collaborator

mvo5 commented Jan 10, 2016

retest this please
@mvo5
Merge remote-tracking branch 'upstream/master' into refactor/snaps-no… 
…-apps

All checks have passed

3 successful checks
@snappy-m-o
Integration tests — Success 54 tests run, 0 skipped, 0 failed.
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage decreased (-0.03%) to 67.595%
Details
a8e0794

mvo5 added a commit that referenced this pull request Jan 12, 2016

@mvo5
Merge pull request #308 from mvo5/refactor/snaps-no-apps 
Rename /apps -> /snaps (same for data, and user-data)
0487a31

@mvo5 mvo5 merged commit 0487a31 into snapcore:master Jan 12, 2016
3 checks passed

3 checks passed

Integration tests Success 54 tests run, 0 skipped, 0 failed.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.03%) to 67.595%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment