commands: Remove some TODOs
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Wed, 11 Apr 2018 08:15:01 +0000 (10:15 +0200)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Wed, 11 Apr 2018 08:16:30 +0000 (10:16 +0200)
See #4598

commands/commands.go
commands/hugo.go
commands/list.go
commands/new.go
commands/new_content_test.go
commands/release.go
commands/server.go

index 2431e5500addf48c780fbcc05ccd2d986bcaa530..aad47abcd0680604454f2ec2cb6dbd065a8afcc2 100644 (file)
@@ -75,22 +75,11 @@ func newBuilderCmd(cmd *cobra.Command) *baseBuilderCmd {
        return bcmd
 }
 
-// TODO(bep) cli refactor need root?
 func (c *baseCmd) flagsToConfig(cfg config.Provider) {
        initializeFlags(c.cmd, cfg)
 }
 
 type hugoCmd struct {
-
-       //cacheDir        string
-       //contentDir      string
-       //layoutDir       string
-       //destination     string
-       //theme           string
-       //themesDir       string
-       //logI18nWarnings bool
-       //disableKinds []string
-
        *baseBuilderCmd
 }
 
index 35b18ae9984e2405704d9a7f035fbfae1e3f3ed7..4639c0f3be68699d2fcf78a6ddb3910d701374e5 100644 (file)
@@ -53,20 +53,6 @@ import (
 
 // TODO(bep) cli refactor consider a exported Hugo() method to fix the API
 
-// Hugo represents the Hugo sites to build. This variable is exported as it
-// is used by at least one external library (the Hugo caddy plugin). We should
-// provide a cleaner external API, but until then, this is it.
-// TODO(bep)  cli refactor remove this
-//var Hugo *hugolib.HugoSites
-
-// Reset resets Hugo ready for a new full build. This is mainly only useful
-// for benchmark testing etc. via the CLI commands.
-// TODO(bep) cli refactor check usage
-func Reset() error {
-       //Hugo = nil
-       return nil
-}
-
 // Execute adds all child commands to the root command HugoCmd and sets flags appropriately.
 func Execute() {
        hugoCmd := newHugoCompleteCmd()
index cf31f2bb475f0ef90856f3542fe83ea463a6f3a8..57a92082cd780490d5ad864dc6358782d6ca3c89 100644 (file)
@@ -150,7 +150,6 @@ expired.`,
                },
        )
 
-       // TODO(bep) cli refactor
        cc.cmd.PersistentFlags().StringVarP(&cc.source, "source", "s", "", "filesystem path to read files relative from")
        cc.cmd.PersistentFlags().SetAnnotation("source", cobra.BashCompSubdirsInDir, []string{})
 
index c78c00bde15dc7a19b60391b59e61e02261d23fb..c088dca9b86683b6686e21843c6371cb30061ab2 100644 (file)
@@ -53,7 +53,6 @@ If archetypes are provided in your theme or site, they will be used.`,
        })
 
        cc.cmd.Flags().StringVarP(&cc.contentType, "kind", "k", "", "content type to create")
-       // TODO(bep) cli refactor
        cc.cmd.PersistentFlags().StringVarP(&cc.source, "source", "s", "", "filesystem path to read files relative from")
        cc.cmd.PersistentFlags().SetAnnotation("source", cobra.BashCompSubdirsInDir, []string{})
        cc.cmd.Flags().StringVar(&cc.contentEditor, "editor", "", "edit new content with this editor, if provided")
@@ -169,7 +168,6 @@ func touchFile(fs afero.Fs, x ...string) {
        }
 }
 
-// TODO(bep) cli refactor => method
 func newContentPathSection(path string) (string, string) {
        // Forward slashes is used in all examples. Convert if needed.
        // Issue #1133
index 212b0ade18711752419b36f11cf01935e2617413..364e0f7839c5502186a8cb718436094ac97e7c17 100644 (file)
@@ -48,7 +48,6 @@ func checkNewSiteInited(fs *hugofs.Fs, basepath string, t *testing.T) {
 }
 
 func TestDoNewSite(t *testing.T) {
-       // TODO(bep) cli refactor
        n := newNewSiteCmd()
        basepath := filepath.Join("base", "blog")
        _, fs := newTestCfg()
index 1846be00dbe76deb60957bbfa20e964b0019998d..a95697e49e93b48ee400625bc3a7f65285cf515e 100644 (file)
@@ -22,11 +22,6 @@ import (
        "github.com/spf13/cobra"
 )
 
-func init() {
-       // TODO(bep) cli refactor
-       //HugoCmd.AddCommand(createReleaser().cmd)
-}
-
 type releaseCommandeer struct {
        cmd *cobra.Command
 
index 5500526e840997e592dbc48a256b8ccb1ca7dd69..1747e17ab757bf0e354cffc51339c6830ee2d9a1 100644 (file)
@@ -85,7 +85,6 @@ of a second, you will be able to save and see your changes nearly instantly.`,
                RunE: cc.server,
        })
 
-       // TODO(bep) cli refactor fields vs strings
        cc.cmd.Flags().IntVarP(&cc.serverPort, "port", "p", 1313, "port on which the server will listen")
        cc.cmd.Flags().IntVar(&cc.liveReloadPort, "liveReloadPort", -1, "port for live reloading (i.e. 443 in HTTPS proxy situations)")
        cc.cmd.Flags().StringVarP(&cc.serverInterface, "bind", "", "127.0.0.1", "interface to which the server will bind")
@@ -230,7 +229,6 @@ func (s *serverCmd) server(cmd *cobra.Command, args []string) error {
        }
 
        c, err := initializeConfig(true, &s.hugoBuilderCommon, s, cfgInit)
-       // TODO(bep) cli refactor
        if err != nil {
                return err
        }
@@ -349,7 +347,6 @@ func (f *fileServer) createEndpoint(i int) (*http.ServeMux, string, string, erro
        return mu, u.String(), endpoint, nil
 }
 
-// TODO(bep) cli refactor
 func (c *commandeer) serve(s *serverCmd) error {
 
        isMultiHost := c.hugo.IsMultihost()