Skip to content

Feat/polish - #4

Merged
patramsey merged 5 commits into
mainfrom
feat/polish
Jun 26, 2026
Merged

Feat/polish#4
patramsey merged 5 commits into
mainfrom
feat/polish

Conversation

@patramsey

Copy link
Copy Markdown
Owner

No description provided.

* Enable additional linters: revive, misspell, and unconvert in .golangci.yml for improved code quality checks.
* Add gofmt as a formatter to enforce consistent code formatting in CI.
* Update go.mod to include golang.org/x/mod v0.30.0, ensuring compatibility with the latest module features.
* Refactor command files for cleaner formatting and improved readability.
* Clean up formatting in various command files to enhance code clarity and maintainability.
* Adjust spacing and alignment in command definitions for consistency across the codebase.
* Update .golangci.yml to include a new revive linter rule for internal API error handling.
* Add package documentation to main.go for better understanding of the CLI's purpose.
* Improve readability in cmdutil.go by clarifying context key comments.
* Refactor command functions in various files to use underscore for unused parameters, enhancing code consistency.
* Update test cases in transport_test.go to ignore unused request parameters, improving clarity.
* Add comments for output format constants in output.go to clarify their purpose.
* Update test server handlers in dns_test.go, url_test.go, and transport_test.go to use underscore for unused request parameters, enhancing readability and consistency across test files.
* Simplify boolean badge handling in list_get.go by removing unnecessary type conversions for domain properties.
* Simplify boolean assignments in runUpdate function of manage.go by removing unnecessary type conversions for domain properties, enhancing code clarity and maintainability.
@patramsey
patramsey merged commit ce34134 into main Jun 26, 2026
2 checks passed
@patramsey
patramsey deleted the feat/polish branch June 26, 2026 03:59
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.

1 participant