Skip to content

Fix public doc linting issues#591

Merged
flin-8 merged 5 commits intomainfrom
frank/linting
Apr 19, 2026
Merged

Fix public doc linting issues#591
flin-8 merged 5 commits intomainfrom
frank/linting

Conversation

@flin-8
Copy link
Copy Markdown
Contributor

@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
Contributor 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
Contributor 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
Contributor 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
Contributor 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
Contributor 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
Contributor 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. 🤷

@flin-8 flin-8 merged commit 83aba44 into main Apr 19, 2026
5 checks passed
@flin-8 flin-8 deleted the frank/linting branch April 19, 2026 23:38
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