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

Create a struct with all of Hugo's config options #10602

Merged
merged 1 commit into from
May 16, 2023

Conversation

bep
Copy link
Member

@bep bep commented Jan 4, 2023

Primary motivation is documentation, but it will also hopefully simplify the code.

OK, this also ended up in a full and long needed restructuring of the commands package with end to end tests built with Go's testscripts. Works great, much easier to reason about, but ... work.

I'll list the commands here to help me get test coverage:

  • hugo
    • basic
    • watch
    • mousetrap
  • completion
    • completion_bash
    • completion_fish
    • completion_powershell
    • completion_zsh
  • config
    • config_mounts
  • convert
    • convert_toJSON
    • convert_toTOML
    • convert_toYAML
  • deploy
  • env
  • gen
    • gen_chromastyles
    • gen_doc
    • gen_man
  • import
    • import_jekyll
  • list
    • list_all
    • list_drafts
    • list_expired
    • list_future
  • mod
    • mod_clean
    • mod_get
    • mod_graph
    • mod_init
    • mod_npm
    • mod_npm_pack
    • mod_tidy
    • mod_vendor
    • mod_verify
  • new
    • new_site
    • new_theme
    • new content
  • server
    • basic
    • change content
    • multihost
    • change config
    • change config add language
    • render to server/disk
  • version
  • release
  • profilers

@bep bep force-pushed the feat/allconfig branch 5 times, most recently from 84a44d0 to 098b09e Compare January 24, 2023 17:10
@bep bep modified the milestones: v0.111.0, v0.112.0 Feb 23, 2023
@bep bep force-pushed the feat/allconfig branch 2 times, most recently from ff96321 to 3ac7939 Compare March 6, 2023 10:27
@bep bep force-pushed the feat/allconfig branch 5 times, most recently from 485aaef to 44b38b0 Compare April 9, 2023 09:21
@bep bep force-pushed the feat/allconfig branch 2 times, most recently from d81ec87 to 1602b0e Compare April 11, 2023 09:16
bep added a commit to bep/hugo that referenced this pull request Apr 12, 2023
Note that the config per language part of this will be handled in gohugoio#10602.

Updates gohugoio#9772
bep added a commit to bep/hugo that referenced this pull request Apr 12, 2023
Note that the config per language part of this will be handled in gohugoio#10602.

Updates gohugoio#9772
bep added a commit that referenced this pull request Apr 12, 2023
Note that the config per language part of this will be handled in #10602.

Updates #9772
@bep bep force-pushed the feat/allconfig branch 4 times, most recently from 424a86f to 3bfc87e Compare April 15, 2023 13:59
@bep bep modified the milestones: v0.112.0, v0.113.0 Apr 15, 2023
@bep bep force-pushed the feat/allconfig branch 21 times, most recently from a717ea5 to 78a2904 Compare May 16, 2023 13:52
Primary motivation is documentation, but it will also hopefully simplify the code.

Also,

* Lower case the default output format names; this is in line with the custom ones (map keys) and how
it's treated all the places. This avoids doing `stringds.EqualFold` everywhere.

Closes gohugoio#10896
Closes gohugoio#10620
@bep bep merged commit 241b21b into gohugoio:master May 16, 2023
6 checks passed
Copy link

This pull request 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 May 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant