ipython / ipython-website

IPython website sources. Any fixes to the website should be done on this repository.
https://ipython.org
50 stars 91 forks source link

Announce 5.1, update home page and docs page #116

Closed takluyver closed 7 years ago

takluyver commented 8 years ago

I went to update the website after making the 5.1 release, and did a few other bits of improvement:

Ping @Carreau @willingc

Carreau commented 8 years ago

+1.

As it removes some link of some of our grants, I'd like the +1 of @fperez or @ellisonbg

willingc commented 8 years ago

@Carreau The links to the grants are only removed from the Announcements section of index.rst and are still available in news.rst.

I would recommend merging this PR. A separate PR could handle any items from news.rst that Brian or Fernando wish to keep in the Announcements section.

+1 to merging. Thanks @takluyver!

fperez commented 8 years ago

Agreed, Announcements is meant to stay small and recent, and we roll anything old over to the reverse-chronological news file. Since these particular items are already in "news", this is just regular announcements cleanup.

+1 to merge as-is, and I don't see a need to move anything old back into announcements.

fperez commented 8 years ago

news should be a complete log, and it goes way back to 2006, which is fine. As long as whenever we drop something from announcements we move it over to news (or check it was already there), we can keep it pruned.

Carreau commented 8 years ago

Ah! Thanks all, not sure how I missed all that!

Thanks again Thomas for getting 5.1 out!

I'll merge once i'm on my computer.

On Aug 14, 2016 16:28, "Fernando Perez" notifications@github.com wrote:

news should be a complete log, and it goes way back to 2006, which is fine. As long as whenever we drop something from announcements we move it over to news (or check it was already there), we can keep it pruned.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ipython/ipython-website/pull/116#issuecomment-239704895, or mute the thread https://github.com/notifications/unsubscribe-auth/AAUez7Ty_oHnmBSndjGJgwVUKbICn15tks5qf6SdgaJpZM4Jjrs5 .

willingc commented 8 years ago

@Carreau No problem. I missed it too on my visual review of the code. I didn't see it until I actually loaded up everything locally. Thanks for merging :D

And thanks @takluyver and the whole team 🎉

Carreau commented 7 years ago

Oops seem like we forgot to merge that.

Thanks @takluyver