Skip to content

Support Jekyll Drafts #28

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

Merged
merged 4 commits into from
Aug 8, 2017
Merged

Support Jekyll Drafts #28

merged 4 commits into from
Aug 8, 2017

Conversation

donavon
Copy link
Contributor

@donavon donavon commented Aug 8, 2017

The PR adds a --drafts command line option that will be passed on to the Jekyll build. I have the use case for doing a draft build for an "internal" blog, but push the non-draft build externally.

If approved, could you please merge and publish a v1.11 as soon as possible? Thx.

@0crat
Copy link
Collaborator

0crat commented Aug 8, 2017

@yegor256 please, pay attention to this pull request

@0crat
Copy link
Collaborator

0crat commented Aug 8, 2017

Job gh:yegor256/jekyll-github-deploy#28 is in scope.

@0crat
Copy link
Collaborator

0crat commented Aug 8, 2017

+15 points just awarded to @donavon, total is +15.

@yegor256
Copy link
Owner

yegor256 commented Aug 8, 2017

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Aug 8, 2017

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit c5bdd85 into yegor256:master Aug 8, 2017
@rultor
Copy link
Collaborator

rultor commented Aug 8, 2017

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 2min)

@yegor256
Copy link
Owner

yegor256 commented Aug 8, 2017

@rultor release, tag is 1.11

@rultor
Copy link
Collaborator

rultor commented Aug 8, 2017

@rultor release, tag is 1.11

@yegor256 OK, I will release it now. Please check the progress here

@rultor
Copy link
Collaborator

rultor commented Aug 8, 2017

@rultor release, tag is 1.11

@yegor256 Done! FYI, the full log is here (took me 2min)

@yegor256
Copy link
Owner

yegor256 commented Aug 8, 2017

@donavon done, thanks for your contribution!

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

Successfully merging this pull request may close these issues.

4 participants