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.

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

Get vex-hugo ready for submission to themes.gohugo.io #1

Merged

Conversation

anthonyfok
Copy link
Contributor

These changes should get the Hugo port of Vex in shape for inclusion on https://themes.gohugo.io/ as requested in gohugoio/hugoThemes#330.

Thanks!

Hugo 0.16 is the first version that supports {{- -}} space trimming
in templates.
- Change non-existent languageCode "en-en" to "en"
- Fix params order for custom_stylesheets and custom_scripts to work
- Fix typo in header: "Belive" → "Believe"
- Remove reference about Themefisher license because Vex for Hugo has
  been relicensed under the MIT License
And make indentation in the source HTML file consistent.
@anthonyfok
Copy link
Contributor Author

anthonyfok commented May 30, 2018

Hi @mehedi-sharif,

If you intend to use CC-BY-3.0 rather than MIT as the license for the Vex (for Hugo) theme, please let me know, and I'll adjust this pull request accordingly.

@mehedi-sharif
Copy link
Contributor

I think CC-BY-3.0 if fine for me rather than MIT. When I first submit my theme to hugo I saw that require to MIT but after visiting few themes I saw that is some themes under CC-BY-3.0. So I think it will be great for me CC-BY-3.0.

@mehedi-sharif mehedi-sharif merged commit 617a08c into themefisher:master May 31, 2018
@anthonyfok
Copy link
Contributor Author

I think CC-BY-3.0 if fine for me rather than MIT. When I first submit my theme to hugo I saw that require to MIT but after visiting few themes I saw that is some themes under CC-BY-3.0. So I think it will be great for me CC-BY-3.0.

Done! Please see Pull Request #2 to change the license to CC-BY-3.0.

nhpp-digitizeafrika pushed a commit to nhpp-digitizeafrika/vex-hugo that referenced this pull request Dec 30, 2020
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.

2 participants