Skip to content

Fix typos#1267

Merged
fizyk merged 2 commits intodbfixtures:mainfrom
lpsinger:typo
Feb 6, 2026
Merged

Fix typos#1267
fizyk merged 2 commits intodbfixtures:mainfrom
lpsinger:typo

Conversation

@lpsinger
Copy link
Contributor

@lpsinger lpsinger commented Jan 29, 2026

Chore that needs to be done:

  • Add newsfragment pipenv run towncrier create [issue_number].[type].rst

Types are defined in the pyproject.toml, issue_number either from issue tracker or the Pull request number

Summary by CodeRabbit

  • Bug Fixes
    • Corrected grammatical and spelling errors in several error messages to improve clarity during troubleshooting.
    • Added a brief changelog entry noting these typo fixes for release tracking.

✏️ Tip: You can customise this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026

Walkthrough

Two exception message strings were corrected for grammar and spelling in the pytest_postgresql package; a new news fragment noting "Fix typos" was added. No functional logic, control flow, or public API changes were made.

Changes

Cohort / File(s) Summary
Error message corrections
pytest_postgresql/executor.py, pytest_postgresql/factories/process.py
Corrected exception text: "Could not found" → "Could not find" and fixed "systenm" → "system" in raised error messages.
Changelog fragment
newsfragments/1267.misc.rst
Added a news fragment recording the typo fixes ("Fix typos").

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit nibbles at a typo trail, 🐇
"Could not find" now sings without fail.
"System" mended, messages are bright,
A tiny hop makes the text just right.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix typos' accurately summarises the main change in the changeset, which involves correcting grammatical and spelling errors across multiple files.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@fizyk fizyk merged commit fc380d4 into dbfixtures:main Feb 6, 2026
51 checks passed
@lpsinger lpsinger deleted the typo branch February 9, 2026 14:30
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