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.

Sign up for GitHub

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

Jump to bottom

Sanity checks for import jekyll #1625

Closed
joneskoo opened this issue Nov 26, 2015 · 4 comments · Fixed by #1664
Closed

Sanity checks for import jekyll #1625

joneskoo opened this issue Nov 26, 2015 · 4 comments · Fixed by #1664
Assignees
@anthonyfok
Labels
Bug Outdated
Milestone
v0.16

Comments

@joneskoo
Copy link

There should be sanity checks for jekyll import:

$ hugo import jekyll . .

WARNING: This will overwrite config in current directory and create a messy directory tree.

For instance this should not be allowed (it creates a very deep structure as it reads its own output).

The import should verify that

  1. the source directory looks like a valid jekyll install (look for config maybe? fail if no posts are found?
  2. target directory is empty or doesn't exist (create it)
@anthonyfok anthonyfok added the Bug label Nov 26, 2015
@anthonyfok
Copy link
Member

Hi @coderzh, wants to look at this? :-)

@anthonyfok anthonyfok added this to the v0.16 milestone Dec 2, 2015
@anthonyfok anthonyfok self-assigned this Dec 2, 2015
@anthonyfok anthonyfok mentioned this issue Dec 2, 2015
Merged
anthonyfok added a commit to anthonyfok/hugo that referenced this issue Dec 3, 2015
@anthonyfok
Add sanity checks for hugo import jekyll
c1b5a48 
Also add `--force` flag copied from `hugo new site`.

Fixes gohugoio#1625
@coderzh
Copy link
Contributor

coderzh commented Dec 3, 2015

Thanks for your commit, @anthonyfok . I'm so busy these days and have too few time to solve this issue.

@anthonyfok
Copy link
Member

You are very welcome, @coderzh. 我被那个 exp/beginner 标签吸引了，所以就边学习边改代码了，嘿嘿。 :-)

bramp pushed a commit to bramp/hugo that referenced this issue Dec 17, 2015
@anthonyfok @bramp
Add sanity checks for hugo import jekyll
34f0f30 
Also add `--force` flag copied from `hugo new site`.

Fixes gohugoio#1625
tychoish pushed a commit to tychoish/hugo that referenced this issue Aug 13, 2017
@anthonyfok @tychoish
Add sanity checks for hugo import jekyll
6ad13c0 
Also add `--force` flag copied from `hugo new site`.

Fixes gohugoio#1625
bep added a commit that referenced this issue Mar 26, 2022
@bep
Squashed 'docs/' changes from a393f4cf4..63386081c
d276e90 
63386081c update cSpell config update
15c76494b Update cSpell custom dictionary (#1694)
34f3167b7 Update image processing (#1625)
7462cc798 fix: pipes in sample code break table creation (#1686)
48736447e Update anchorize.md
2ff0bd10b netlify: Hugo 0.95.0
0fc1d21b2 Update configuration.md
41855e372 Fix #1682
8c663433e Update related.md
7aa072eab netlify: Hugo 0.94.2
1682c7ee7 Update render-hooks.md
ce1283cc4 Move the Render Hooks doc to its own page
bbbbfbfc6 Update configuration-markup.md
92d91a316 Update configuration-markup.md
2e8068823 Update configuration-markup.md
ff2dbca60 Update configuration-markup.md
89d8e5d65 Add code block documenation
e993539f0 Update shortcodes.md
c1b28dbfe netlify: Hugo 0.94.1
81b8c9b83 Merge branch 'tempv0.94.1'
4763b3d50 docs: Regenerate CLI docs
b18463971 netlify: Bump to Hugo 0.94.0
4152ebc1d Merge branch 'tempv0.94.0'
ba3a11ac2 docs: Regenerate docshelper
e64016d13 docs: Regenerate docshelper
29180e4d2 add `.html` suffix to partial usage and references
3213e00f2 Docs tidy-up
6cfcae4b7 docs: Regenerate CLI docs
8a6cd0b4d docs: Regenerate docshelper
b20ab262f Merge commit 'd706529720b3b2ccb99719ccd578062ca25a0cc2'

git-subtree-dir: docs
git-subtree-split: 63386081c55de6a7f97adde564a9cfc2ad326119
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@anthonyfok anthonyfok

Labels
Bug Outdated
Projects
None yet
Milestone
v0.16
Development

Successfully merging a pull request may close this issue.

Add sanity checks for hugo import jekyll anthonyfok/hugo
4 participants
@spf13 @joneskoo @coderzh @anthonyfok