Skip to content
This repository has been archived by the owner on Jul 5, 2021. It is now read-only.

Broken Themes on the Hugo Showcase #867

Closed
digitalcraftsman opened this issue Jun 12, 2020 · 21 comments
Closed

Broken Themes on the Hugo Showcase #867

digitalcraftsman opened this issue Jun 12, 2020 · 21 comments
Labels

Comments

@digitalcraftsman
Copy link
Member

digitalcraftsman commented Jun 12, 2020

This issue is meant for author notification and not discussion.

The following themes broke for various reasons and need to be fixed or will be removed. Theme authors should know how to fix their themes based on the errors logs below.

If theme authors do not know how to fix their themes then they should ask at the dedicated Hugo Support Forum: https://discourse.gohugo.io/

When a fix is issued then authors should ping me in the corresponding commit with @digitalcraftsman or respond to this thread, so that I get notified. Themes will be updated automatically every three days.


@gohugoio gohugoio locked and limited conversation to collaborators Jun 12, 2020
@digitalcraftsman digitalcraftsman pinned this issue Jun 12, 2020
@gohugoio gohugoio unlocked this conversation Jun 21, 2020
@stale stale bot added the Stale label Jul 18, 2020
digitalcraftsman added a commit that referenced this issue Jul 18, 2020
digitalcraftsman added a commit that referenced this issue Jul 18, 2020
digitalcraftsman added a commit that referenced this issue Jul 18, 2020
@digitalcraftsman
Copy link
Member Author

digitalcraftsman commented Jul 18, 2020

The following themes have been removed:

@MunifTanjim
Copy link

Minimo should be fixed: MunifTanjim/minimo@594144f 🤔 Is it still broken?

@digitalcraftsman
Copy link
Member Author

Thank you for the fix Munif. I've updated the themes and Minimo's demo is running fine again.

@kaushalmodi
Copy link

@digitalcraftsman I appreciate your work on keeping the Hugo themes clean. Thanks for this notification.

@so1ve
Copy link

so1ve commented Jul 18, 2020

Hello, isn't hugo-theme-tony fixed in ThemeTony/hugo-theme-tony@e01e218? 😢

@digitalcraftsman
Copy link
Member Author

@FFRaycoder let's continue this discussion at ThemeTony/hugo-theme-tony#7

digitalcraftsman added a commit that referenced this issue Jul 18, 2020
so1ve added a commit to ThemeTony/hugo-theme-tony that referenced this issue Jul 18, 2020
so1ve added a commit to ThemeTony/hugo-theme-tony that referenced this issue Jul 18, 2020
so1ve added a commit to ThemeTony/hugo-theme-tony that referenced this issue Jul 18, 2020
so1ve added a commit to ThemeTony/hugo-theme-tony that referenced this issue Jul 18, 2020
@Shadow53
Copy link

Feel free to remove Midnight for the time being. I'm in the middle of a major refactoring and will resubmit once that's done.

digitalcraftsman added a commit that referenced this issue Jul 29, 2020
@kc0bfv
Copy link

kc0bfv commented Jul 31, 2020 via email

@koirand
Copy link

koirand commented Aug 26, 2020

Thanks for the notification, I've fixed the broken theme.

@panr
Copy link

panr commented Sep 6, 2020

@digitalcraftsman hugo-theme-hello-friend and hugo-theme-terminal are fixed. The problem should disappear.

@digitalcraftsman
Copy link
Member Author

Thank you Radek 👍

@knadh
Copy link

knadh commented Sep 7, 2020

@digitalcraftsman unable to reproduce the issue on hugo-ink locally with v0.74.3 on Ubuntu.

$ hugo serve --themesDir ../.. -t hugo-ink

                   | EN  
-------------------+-----
  Pages            | 31  
  Paginator pages  |  1  
  Non-page files   |  0  
  Static files     |  5  
  Processed images |  0  
  Aliases          |  1  
  Sitemaps         |  1  
  Cleaned          |  0  

Built in 16 ms
Watching for changes in $PATH/hugo-ink/{archetypes,exampleSite,layouts,static}
Watching for config changes in $PATH/hugo-ink/exampleSite/config.toml
Environment: "development"
Serving pages from memory
Running in Fast Render Mode. For full rebuilds on change: hugo server --disableFastRender
Web Server is available at http://localhost:1313/ (bind address 127.0.0.1)
Press Ctrl+C to stop

@digitalcraftsman
Copy link
Member Author

digitalcraftsman commented Sep 7, 2020

That's because the error only occurs with the default content for demos privided by https://github.com/gohugoio/hugoBasicExample. The ambiguity occured (here) because one of the default content files was tagged and categorized with themes.

I've removed the corresponding tag such that this ambiguity doesn't occur with https://github.com/gohugoio/hugoBasicExample. But be aware that this isn't fixing future instances (e.g. of your theme's users) in which such an ambiguity can't be resolved by Hugo.

Nonetheless, the demo runs fine now and I'll remove your theme from the list above.

@capnfabs
Copy link

capnfabs commented Jan 1, 2021

Hi @digitalcraftsman, paperesque should be fixed as of this commit: capnfabs/paperesque@d586157

Thanks for the notification and please let me know if there are any additional issues.

@digitalcraftsman
Copy link
Member Author

Thanks for you fix. In your theme repository I see a mainline and master branch. Your linked commit is present on the mainline branch, whereas the Hugo theme site uses the master branch as source for your theme's demo. Which of them should be used for the demo?

@capnfabs
Copy link

capnfabs commented Jan 2, 2021

@digitalcraftsman The mainline branch is correct; I've since deleted the master branch to make it clearer. Note also that the mainline branch is the default on the repo, which might mean it works without changes? At least, the following works on my machine without changes to the hugoThemes repo:

 git clone [email protected]:gohugoio/hugoThemes.git
cd hugoThemes/_script
./reviewTheme.sh [email protected]:capnfabs/paperesque.git

Sorry for not being clear about this initially and thanks for your support!

@digitalcraftsman
Copy link
Member Author

digitalcraftsman commented Jan 2, 2021

Changing the source branch for the theme demo isn't that difficult. Depending on your answer I had to make this change or not. Nonetheless, your theme has been updated and the demo is running again. I'll remove your theme from the first comment of this thread in a minute.

@matsuyoshi30
Copy link

@digitalcraftsman Thanks for the notification and sorry for the delay. harbor should be fixed at matsuyoshi30/harbor@cae4c31. If it is still broken, please let me know!

@digitalcraftsman
Copy link
Member Author

Thank you fixing the issue. Your theme has been updated and the demo is working again. I'll remove your theme from the first comment of this thread in a minute.

@0xforever9
Copy link

https://github.com/4ever9/less
hugo(extended) caused this error. You should use npm i to install dependency.

@digitalcraftsman

@digitalcraftsman
Copy link
Member Author

Hi @4ever9,

the demos on www.themes.gohugo.io are build with the "basic" version of Hugo (on purpose), thus stylesheets, JS files etc. would have to be prebuild and pushed to a theme's Git repository.

The workaround for cases, where a theme's couldn't be hosted on www.themes.gohugo.io (e.g. special theme requirements/dependencies), theme authors self-hosted the theme demo. Usually, they would link the demo using the homepage variable in theme.toml or add a link at the top of their theme's README. The themes are still listed on www.themes.gohugo.io though! The docsy theme is such a case.

Even if you decide aginst self-hosting, your theme would still be listed on www.themes.gohugo.io, just with the demo (button) disabled.

@bep bep closed this as completed Jul 4, 2021
@digitalcraftsman digitalcraftsman unpinned this issue Jul 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests