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

Remove migrators #793

Merged
merged 6 commits into from Feb 3, 2013

Conversation

Projects
None yet
5 participants
@parkr
Member

parkr commented Jan 30, 2013

Code to complete #787.

Basically, we want to move migrators out of the main Jekyll project and release them as their own gem: jekyll-import. They've been extracted and moved to jekyll/jekyll-import.

@mattr-

This comment has been minimized.

Member

mattr- commented Jan 31, 2013

👍

nuke 'em

@kevinSuttle

This comment has been minimized.

kevinSuttle commented Feb 2, 2013

👍

bin/jekyll Outdated
require 'jekyll-import'
rescue
puts "You must install the 'jekyll-import' gem before continuing."
exit(-1)

This comment has been minimized.

@tombell

tombell Feb 2, 2013

Contributor

Use abort "You must install the 'jekyll-import' gem before continuing.'".

This comment has been minimized.

@parkr

parkr Feb 3, 2013

Member

Nice one! I'll update that now. Totally forgot about abort.

@tombell

This comment has been minimized.

Contributor

tombell commented Feb 2, 2013

👍

parkr added a commit that referenced this pull request Feb 3, 2013

Merge pull request #793 from mojombo/remove-migrators
Remove migrators - move to jekyll-import gem.

@parkr parkr merged commit 9ea1bec into master Feb 3, 2013

parkr added a commit that referenced this pull request Feb 3, 2013

@parkr parkr deleted the remove-migrators branch Dec 15, 2013

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

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