Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update projects readme files to link to MyGet Gallery #5

Open
gep13 opened this issue Apr 13, 2017 · 6 comments
Open

Update projects readme files to link to MyGet Gallery #5

gep13 opened this issue Apr 13, 2017 · 6 comments

Comments

@gep13
Copy link
Member

gep13 commented Apr 13, 2017

http://myget.org/gallery/cake-contrib

As suggested by @Jericho

@Jericho
Copy link
Member

Jericho commented Apr 13, 2017

Here is an example of what I suggest:

| |Stable|Pre-release|
|:--:|:--:|:--:|
|GitHub Release|-|[![GitHub release](https://img.shields.io/github/release/cake-contrib/Cake.SendGrid.svg)](https://github.com/cake-contrib/Cake.SendGrid/releases/latest)|
|Package|[![NuGet](https://img.shields.io/nuget/v/Cake.SendGrid.svg)](https://www.nuget.org/packages/Cake.SendGrid)|[![MyGet](https://img.shields.io/myget/cake-contrib/vpre/Cake.SendGrid.svg)](http://myget.org/feed/cake-contrib/package/nuget/Cake.SendGrid)|

Which results in the following table:
image

By the way, I'm not sure what the issue is with the "GitHub release" section of the table. It always shows the "none" badge for some reason.

@gep13
Copy link
Member Author

gep13 commented Apr 14, 2017

@Jericho yeah, I think this entire section needs quite a bit of work.

The url:

https://github.com/cake-contrib/Cake.SendGrid/releases/latest

will only return something if you remove all the pre-release flags from here:

https://github.com/cake-contrib/Cake.SendGrid/releases

I guess the pre-release flag should only be set when the tag includes a -beta or -rc addition.

I think the shield should be moved from the Pre-release column to the Stable column, and perhaps simply have n/a in the pre-release column.

Thoughts?

@Jericho
Copy link
Member

Jericho commented Apr 14, 2017

  • Removing the pre-release flag on GitHub: I'm fine with that
  • Moving the badge to the "Stable" column: makes sense
  • "n/a" in the pre-release column: fine with me

It occurs to me: do we really need a table to present this information? What about 3 simple bullet points like this:

  • Release Notes: <badge to GitHub releases>
  • Package: <badge to Nuget package>
  • Pre-release package: <badge to myget package>

@Jericho
Copy link
Member

Jericho commented Apr 14, 2017

If we want to keep the "table" format, what about something like this:

Release Notes GitHub release
Package NuGet
Pre-release package MyGet

@pascalberger
Copy link
Member

IMHO there's a difference between pre-release packages from nuget.org and packages from myget. pre-release packages from nuget.org are official pre-relases (beta) targeted at users of the addin for testing purposes before a final release. Packages from myget I consider unstable development version, which can be useful for testing specific features or bugs, but are not supposed to be used by a wider audience.

@gep13
Copy link
Member Author

gep13 commented Apr 14, 2017

@pascalberger yes, I don't disagree with this. Perhaps since we can't easily get that information from a single URL on GitHub, perhaps Pre-release package becomes CI Build?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants