Skip to content

Conversation

@apoclyps
Copy link
Contributor

What's Changed

This pull request introduces several improvements focused on code quality, documentation, and typo corrections. The main changes include adding a pull request template, integrating a spelling checker into the pre-commit workflow, and correcting various spelling mistakes across the codebase.

Documentation and Process Improvements

  • Added a standardized pull request template in .github/PULL_REQUEST_TEMPLATE.md to guide contributors in documenting their changes.

Tooling Enhancements

  • Integrated the typos spelling checker into the pre-commit configuration via .pre-commit-config.yaml and added a custom dictionary in .typos.toml to handle project-specific terms.

Spelling and Typo Corrections

  • Fixed multiple spelling errors throughout the codebase and documentation, including words like "separated", "configuration", "throttled", and "explicitly".

@apoclyps apoclyps self-assigned this Jan 23, 2026
@apoclyps apoclyps requested a review from a team as a code owner January 23, 2026 16:47
Copilot AI review requested due to automatic review settings January 23, 2026 16:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request introduces typos checking into the CI pipeline to catch spelling errors automatically, while also fixing multiple spelling mistakes that already exist in the codebase. The PR includes tooling configuration, spelling corrections, and a standardized pull request template.

Changes:

  • Added typos checker to pre-commit hooks and GitHub Actions CI workflow
  • Fixed spelling errors throughout the codebase including "explicitly", "throttled", "separator", "configuration", "separated", and "automatically"
  • Added pull request template to standardize contribution documentation

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.typos.toml Configuration file for typos checker with custom dictionary to handle project-specific terms like proper names and intentional partial words
.pre-commit-config.yaml Added typos hook (v1.42.0) to pre-commit configuration
.github/workflows/lint.yml Added typos CI job using GitHub Actions (v1.29.5)
.github/PULL_REQUEST_TEMPLATE.md Added standardized PR template for contributors
cloudsmith_cli/core/tests/test_init.py Fixed typo: "explcitly" → "explicitly"
cloudsmith_cli/core/ratelimits.py Fixed typo: "throtted" → "throttled" (important for API key matching)
cloudsmith_cli/cli/tests/commands/test_repos.py Fixed typo: "seperator" → "separator"
cloudsmith_cli/cli/config.py Fixed typo: "configuratin" → "configuration"
cloudsmith_cli/cli/commands/metrics/packages.py Fixed typo: "seperated" → "separated"
cloudsmith_cli/cli/commands/metrics/entitlements.py Fixed typo: "seperated" → "separated"
CHANGELOG.md Fixed typo: "automaticlly" → "automatically"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@apoclyps apoclyps force-pushed the kyleharrison/no-ticket/add-support-for-typos branch from 53a5866 to fcc5702 Compare January 23, 2026 16:51
@apoclyps apoclyps force-pushed the kyleharrison/no-ticket/add-support-for-typos branch from fcc5702 to da5f494 Compare January 23, 2026 16:59
Copy link
Member

@BartoszBlizniak BartoszBlizniak left a comment

Choose a reason for hiding this comment

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

nit-pick: Should we also include a template for issues?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants