7digital / SevenDigital.Api.Wrapper

A fluent c# wrapper for the 7digital API
MIT License
16 stars 29 forks source link

There is no Release on GitHub for v5.0.0 #216

Closed bettiolo closed 9 years ago

bettiolo commented 9 years ago

Can you create a release with the change log?

Thanks.

AnthonySteele commented 9 years ago

What does that mean?

7digital-Api-Wrapper-Notifications commented 9 years ago

Marco's talking about creating a release here:

https://github.com/7digital/SevenDigital.Api.Wrapper/releases

I guess this is related to what were were chatting about. When releasing a version on nuget we need to drive it from github, i.e. create a "release" first and have the nuget package build from that branch within Teamcity (or similar).

That way nuget packages are always tied to a specific stage of the code and make it easier for people to track changes etc (by checking the commits).

For now, we can retrospectively create a release on gihub for 5.0.0 though can't we, with some basic release notes about the splitting of the schema and the wrapper?

On 8 December 2014 at 16:10, Anthony Steele notifications@github.com wrote:

What does that mean?

— Reply to this email directly or view it on GitHub https://github.com/7digital/SevenDigital.Api.Wrapper/issues/216#issuecomment-66138573 .

Greg Sochanik Developer Api Device Integration

T: +44(0)7968062483 E: greg.sochanik@7digital.com W: www.7digital.com http://www.7digital/

@gregsochanik

This email, including attachments, is private and confidential. If you have received this email in error please notify the sender and delete it from your system. Emails are not secure and may contain viruses. No liability can be accepted for viruses that might be transferred by this email or any attachment. Any unauthorised copying of this message or unauthorised distribution and publication of the information contained herein are prohibited.

7digital Limited. Registered office: 69 Wilson Street, London EC2A 2BB. Registered in England and Wales. Registered No. 04843573.

AnthonySteele commented 9 years ago

Sounds like it's worth doing.

bettiolo commented 9 years ago

You did it for the previous releases like https://github.com/7digital/SevenDigital.Api.Wrapper/tree/v4.0.0, I was looking at what changed for this new one and I couldn't find a release tag.

AnthonySteele commented 9 years ago

Yep, I'll have a look when I have time. I suggest that we also make notes on how to do this in future so that we do it consistently.

7digital-Api-Wrapper-Notifications commented 9 years ago

No probs Marco, Anthony, have you got time to set up some release notes for 5.0.0?

I'll raise a new issue for what I was talking about (TC nuget releases linked to github releases) and look at it during the release freeze.

On 8 December 2014 at 22:47, Marco Bettiolo notifications@github.com wrote:

You did it for the previous releases like https://github.com/7digital/SevenDigital.Api.Wrapper/tree/v4.0.0, I was looking at what changed for this new one and I couldn't find a release tag.

— Reply to this email directly or view it on GitHub https://github.com/7digital/SevenDigital.Api.Wrapper/issues/216#issuecomment-66202793 .

Greg Sochanik Developer Api Device Integration

T: +44(0)7968062483 E: greg.sochanik@7digital.com W: www.7digital.com http://www.7digital/

@gregsochanik

This email, including attachments, is private and confidential. If you have received this email in error please notify the sender and delete it from your system. Emails are not secure and may contain viruses. No liability can be accepted for viruses that might be transferred by this email or any attachment. Any unauthorised copying of this message or unauthorised distribution and publication of the information contained herein are prohibited.

7digital Limited. Registered office: 69 Wilson Street, London EC2A 2BB. Registered in England and Wales. Registered No. 04843573.

AnthonySteele commented 9 years ago

I will base the release notes off the detailed email that I sent out

gregsochanik commented 9 years ago

Right - logged in now :-)

Cool, I've raised the issue here #217

gregsochanik commented 9 years ago

Release set up for v5.0.0

https://github.com/7digital/SevenDigital.Api.Wrapper/releases/tag/v5.0.0