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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add post about Jekyll Admin initial release #5291

Merged
merged 3 commits into from Aug 25, 2016

Conversation

Projects
None yet
8 participants
@mertkahyaoglu
Member

mertkahyaoglu commented Aug 24, 2016

@jekyll/gsoc, anything you'd like to add ? 馃槃

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 24, 2016

Member

This is AMAZING! LGTM! !!!

Member

parkr commented Aug 24, 2016

This is AMAZING! LGTM! !!!

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 24, 2016

Contributor

Is it ready yet? We've been unable to get it to boot on our Windows machines where it's more useful.

Contributor

envygeeks commented Aug 24, 2016

Is it ready yet? We've been unable to get it to boot on our Windows machines where it's more useful.

@benbalter

This comment has been minimized.

Show comment
Hide comment
@benbalter

benbalter Aug 24, 2016

Contributor

Is it ready yet? We've been unable to get it to boot on our Windows machines where it's more useful.

It should be. Have you tried v0.1.0? See jekyll/jekyll-admin#175 and jekyll/jekyll-admin#181 (regarding development on Windows, installation should be fine).

We've been testing on Windows and @codegaze (a Window user) has been helping us and submitting a handful of PRs (both Windows and not Windows related to improve and document the experience).

Contributor

benbalter commented Aug 24, 2016

Is it ready yet? We've been unable to get it to boot on our Windows machines where it's more useful.

It should be. Have you tried v0.1.0? See jekyll/jekyll-admin#175 and jekyll/jekyll-admin#181 (regarding development on Windows, installation should be fine).

We've been testing on Windows and @codegaze (a Window user) has been helping us and submitting a handful of PRs (both Windows and not Windows related to improve and document the experience).

@benbalter

This comment has been minimized.

Show comment
Hide comment
@benbalter

benbalter Aug 24, 2016

Contributor

LGTM (pending @envygeeks's question above).

We may want to bump the date until tomorrow? (it's already the 25th in Turkey).

Contributor

benbalter commented Aug 24, 2016

LGTM (pending @envygeeks's question above).

We may want to bump the date until tomorrow? (it's already the 25th in Turkey).

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 24, 2016

Contributor

LGTM.

Contributor

envygeeks commented Aug 24, 2016

LGTM.

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 24, 2016

Contributor

Thanks for answering @benbalter if you've got windows testers who can get it working then we do need to update and tell our Windows users to try again. It was about a month ago we first tried it.

Contributor

envygeeks commented Aug 24, 2016

Thanks for answering @benbalter if you've got windows testers who can get it working then we do need to update and tell our Windows users to try again. It was about a month ago we first tried it.

@benbalter

This comment has been minimized.

Show comment
Hide comment
@benbalter

benbalter Aug 24, 2016

Contributor

Lol @jekyllbot.

LGTM.

Contributor

benbalter commented Aug 24, 2016

Lol @jekyllbot.

LGTM.

@mertkahyaoglu

This comment has been minimized.

Show comment
Hide comment
@mertkahyaoglu

mertkahyaoglu Aug 25, 2016

Member

@benbalter Not sure what timezone I should use.

Member

mertkahyaoglu commented Aug 25, 2016

@benbalter Not sure what timezone I should use.

@jaybe-jekyll

This comment has been minimized.

Show comment
Hide comment
@jaybe-jekyll

jaybe-jekyll Aug 25, 2016

Member

Awesomeness. So excited about this! Nice!

Member

jaybe-jekyll commented Aug 25, 2016

Awesomeness. So excited about this! Nice!

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 25, 2016

Contributor

LGTM.

Contributor

envygeeks commented Aug 25, 2016

LGTM.

@mertkahyaoglu

This comment has been minimized.

Show comment
Hide comment
@mertkahyaoglu

mertkahyaoglu Aug 25, 2016

Member

I have no idea why those checks are failing.

Member

mertkahyaoglu commented Aug 25, 2016

I have no idea why those checks are failing.

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 25, 2016

Contributor

We ignore Appveyor for now, and Travis sometimes have random failures I am personally (I don't know if anybody else has tried) to pin down and replicate locally on my own Linux machine.

Contributor

envygeeks commented Aug 25, 2016

We ignore Appveyor for now, and Travis sometimes have random failures I am personally (I don't know if anybody else has tried) to pin down and replicate locally on my own Linux machine.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 25, 2016

Member

@benbalter Not sure what timezone I should use.

Up to you! Probably best to use your own timezone. 馃槃

Member

parkr commented Aug 25, 2016

@benbalter Not sure what timezone I should use.

Up to you! Probably best to use your own timezone. 馃槃

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 25, 2016

Member

@benbalter Those LGTM's are commit-focused so if you update the PR it is re-reviewed. The intention anyway. 馃槃

Member

parkr commented Aug 25, 2016

@benbalter Those LGTM's are commit-focused so if you update the PR it is re-reviewed. The intention anyway. 馃槃

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 25, 2016

Contributor

@parkr that is a pretty nice feature.

On Wed, Aug 24, 2016, 7:34 PM Parker Moore notifications@github.com wrote:

@benbalter https://github.com/benbalter Those LGTM's are commit-focused
so if you update the PR it is re-reviewed. The intention anyway. 馃槃


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#5291 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGFs8YthU3BXHUt_Fu9SY-KTHZ6BK3oks5qjOMpgaJpZM4JsiMw
.

Contributor

envygeeks commented Aug 25, 2016

@parkr that is a pretty nice feature.

On Wed, Aug 24, 2016, 7:34 PM Parker Moore notifications@github.com wrote:

@benbalter https://github.com/benbalter Those LGTM's are commit-focused
so if you update the PR it is re-reviewed. The intention anyway. 馃槃


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#5291 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGFs8YthU3BXHUt_Fu9SY-KTHZ6BK3oks5qjOMpgaJpZM4JsiMw
.

@ashmaroli

This comment has been minimized.

Show comment
Hide comment
@ashmaroli

ashmaroli Aug 25, 2016

Member

@benbalter if you've got windows testers who can get it working then we do need to update and tell our Windows users to try again.

@envygeeks, I have tried jekyll-admin locally (on Windows). It works as documented, if that was what you were enquiring..
@mertkahyaoglu and his mentors, Thank you so much for adding the UI interface. I 鉂わ笍 it

P.S. I'd be great if the Gemfile generated by jekyll new had this plugin on by default.

Member

ashmaroli commented Aug 25, 2016

@benbalter if you've got windows testers who can get it working then we do need to update and tell our Windows users to try again.

@envygeeks, I have tried jekyll-admin locally (on Windows). It works as documented, if that was what you were enquiring..
@mertkahyaoglu and his mentors, Thank you so much for adding the UI interface. I 鉂わ笍 it

P.S. I'd be great if the Gemfile generated by jekyll new had this plugin on by default.

@codegaze

This comment has been minimized.

Show comment
Hide comment
@codegaze

codegaze Aug 25, 2016

@benbalter @envygeeks Yeap, gem installation and basic use are fine on Windows.
I also changed the title to jekyll/jekyll-admin#175 "Windows local development installation bugs" to avoid any confusion with the gem installation.

codegaze commented Aug 25, 2016

@benbalter @envygeeks Yeap, gem installation and basic use are fine on Windows.
I also changed the title to jekyll/jekyll-admin#175 "Windows local development installation bugs" to avoid any confusion with the gem installation.

@mertkahyaoglu

This comment has been minimized.

Show comment
Hide comment
@mertkahyaoglu

mertkahyaoglu Aug 25, 2016

Member

Hopefully this will be the last commit and then we can finally publish it 馃槵 Sorry for requesting too many LGTMs.

Member

mertkahyaoglu commented Aug 25, 2016

Hopefully this will be the last commit and then we can finally publish it 馃槵 Sorry for requesting too many LGTMs.

@benbalter

This comment has been minimized.

Show comment
Hide comment
@benbalter

benbalter Aug 25, 2016

Contributor

LGTM x3

Contributor

benbalter commented Aug 25, 2016

LGTM x3

@parkr parkr added the documentation label Aug 25, 2016

@benbalter

This comment has been minimized.

Show comment
Hide comment
@benbalter

benbalter Aug 25, 2016

Contributor

LGTM.

Contributor

benbalter commented Aug 25, 2016

LGTM.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 25, 2016

Member

LGTM.

Member

parkr commented Aug 25, 2016

LGTM.

@benbalter

This comment has been minimized.

Show comment
Hide comment
@benbalter

benbalter Aug 25, 2016

Contributor

@jekyllbot: merge +site

Contributor

benbalter commented Aug 25, 2016

@jekyllbot: merge +site

@jekyllbot jekyllbot merged commit 59b8f2d into jekyll:master Aug 25, 2016

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jekyll/lgtm Approved by @benbalter and @parkr.

jekyllbot added a commit that referenced this pull request Aug 25, 2016

@parkr

This comment has been minimized.

Show comment
Hide comment

@mertkahyaoglu mertkahyaoglu deleted the mertkahyaoglu:jekyll-admin-post branch Aug 26, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment