Skip to content

Fix public doc linting issues#591

Open
flin-8 wants to merge 5 commits intomainfrom
frank/linting
Open

Fix public doc linting issues#591
flin-8 wants to merge 5 commits intomainfrom
frank/linting

Conversation

@flin-8
Copy link
Copy Markdown

@flin-8 flin-8 commented Apr 17, 2026

@flin-8 flin-8 self-assigned this Apr 17, 2026
Comment thread cmd/gen-docs/main.go

{{.Command.Long}}
` + "\n```" + `{{define "T1"}}Usage:{{if .Runnable}}
` + "\n```text" + `{{define "T1"}}Usage:{{if .Runnable}}
Copy link
Copy Markdown
Author

@flin-8 flin-8 Apr 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MD040 fenced-code-language - Fenced code blocks should have a language specified

Comment thread cmd/gen-docs/main.go
- [Octopus CLI](/docs/octopus-rest-api/cli)
- [Creating API keys](/docs/octopus-rest-api/how-to-create-an-api-key)`
- [Creating API keys](/docs/octopus-rest-api/how-to-create-an-api-key)
`
Copy link
Copy Markdown
Author

@flin-8 flin-8 Apr 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MD047 single-trailing-newline - Files should end with a single newline character

Comment thread cmd/gen-docs/main.go
## Commands {#octopusCommandLine-Commands}

` + "\n`octopus` supports the following commands:\n" +
` + "`octopus` supports the following commands:" +
Copy link
Copy Markdown
Author

@flin-8 flin-8 Apr 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MD012 no-multiple-blanks - Multiple consecutive blank lines

Short: "Create an AWS account",
Long: "Create an AWS account in Octopus Deploy",
Example: heredoc.Docf("$ %s account aws create", constants.ExecutableName),
Example: heredoc.Docf("%s account aws create", constants.ExecutableName),
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ %[1]s project variable update
%[1]s project variable list "Deploy Web App"
%[1]s project variable view --name "DatabaseName" --project Deploy
%[1]s project variable update
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MD014 commands-show-output - Dollar signs used before commands without showing output

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zentron if we think it's important to keep the $ at the front, then we can override the linting rule

Copy link
Copy Markdown
Contributor

@zentron zentron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scanning through the changes it all looks reasonable, im assuming the format must have changed at some point since it was last generated. 🤷

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