Releasing 2.1 #286

Merged
merged 1 commit into from Feb 3, 2016

Conversation

Projects
None yet
3 participants
Collaborator

sergiusens commented Feb 2, 2016

No description provided.

@sergiusens sergiusens changed the title from Release changelog for 2.1 to Releasing 2.1 Feb 2, 2016

debian/changelog
@@ -1,3 +1,52 @@
+snapcraft (2.0.1ubuntu1) xenial; urgency=medium
@kyrofa

kyrofa Feb 2, 2016

Member

Not 2.1?

@sergiusens

sergiusens Feb 2, 2016

Collaborator

argh, I thought about it but did not change it :-/

debian/changelog
+
+ [ Reto Breitenmoser ]
+ * tar-content plugin: Add capability to unpack into a specific
+ destination directory LP: #1478055
@kyrofa

kyrofa Feb 2, 2016

Member

Consider surrounding the bug reference with parentheses for consistency.

@sergiusens

sergiusens Feb 2, 2016

Collaborator

I wonder how this wasn't automatic

debian/changelog
+ * added support for uploading snap packages directly to the Ubuntu
+ Store. LP: #1538657
+ * make sure otp is included in login request only if it's not
+ empty LP: #1539208
@kyrofa

kyrofa Feb 2, 2016

Member

Should we capitalize these to make them all pretty-like?

@sergiusens

sergiusens Feb 2, 2016

Collaborator

yes

Collaborator

sergiusens commented Feb 2, 2016

@kyrofa thanks and done

debian/changelog
+snapcraft (2.1) xenial; urgency=medium
+
+ [ Leo Arias ]
+ * Run the tests in a xenial LXC. LP: #1532213
@kyrofa

kyrofa Feb 2, 2016

Member

One more here. Why is this happening? Looking at the commit, the only thing that seems to differ is that there's no blank line after the title.

@sergiusens

sergiusens Feb 2, 2016

Collaborator

I have no idea!

@sergiusens

sergiusens Feb 2, 2016

Collaborator

to be fair, gbp is not as good as bzr bd so that may be an indication.

@kyrofa

kyrofa Feb 2, 2016

Member

Blasphemer!

@elopio

elopio Feb 2, 2016

Member

ugh, I was so close this time to get all my commit messages perfect. Damn!

debian/changelog
+ * Added support for uploading snap packages directly to the Ubuntu
+ Store. LP: #1538657
+ * Make sure otp is included in the login request only if it's not
+ empty LP: #1539208
@kyrofa

kyrofa Feb 2, 2016

Member

Still got these.

@sergiusens

sergiusens Feb 2, 2016

Collaborator

darn, I only looked under the one you mentioned thinking it was only one occurrence :-)

Member

kyrofa commented Feb 2, 2016

There, you have my 👍 now.

sergiusens added a commit that referenced this pull request Feb 3, 2016

@sergiusens sergiusens merged commit 5bf95bf into snapcore:master Feb 3, 2016

2 checks passed

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

@sergiusens sergiusens deleted the sergiusens:changelog/2.1 branch Feb 3, 2016

smoser pushed a commit to smoser/snapcraft that referenced this pull request Sep 14, 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