Skip to content

fix: correct typos in code comments#1387

Merged
vmihailenco merged 1 commit into
uptrace:masterfrom
maxtaran2010:fix/typos-in-comments
Jul 17, 2026
Merged

fix: correct typos in code comments#1387
vmihailenco merged 1 commit into
uptrace:masterfrom
maxtaran2010:fix/typos-in-comments

Conversation

@maxtaran2010

Copy link
Copy Markdown

Fix several spelling mistakes in Go doc comments and inline comments.

Changes:

  • constaint -> constraint in migrate/auto.go
  • lenght -> length in migrate/operations.go and dialect/pgdialect/sqltype.go
  • colums -> columns in migrate/sqlschema/database.go
  • overriden -> overridden in migrate/sqlschema/inspector.go and internal/dbtest/migrate_test.go
  • embeded -> embedded in migrate/sqlschema/migrator.go
  • configutable -> configurable in extra/bunexp/resolver.go
  • inpection -> inspection in internal/dbtest/migrate_test.go

All fixes are in comments only — no logic, API, or behavior is changed.

Fix several spelling mistakes in Go doc comments and inline comments
across migrate, sqlschema, pgdialect, and bunexp packages.

- constaint -> constraint (migrate/auto.go)
- lenght -> length (migrate/operations.go, dialect/pgdialect/sqltype.go)
- colums -> columns (migrate/sqlschema/database.go)
- overriden -> overridden (migrate/sqlschema/inspector.go, internal/dbtest/migrate_test.go)
- embeded -> embedded (migrate/sqlschema/migrator.go)
- configutable -> configurable (extra/bunexp/resolver.go)
- inpection -> inspection (internal/dbtest/migrate_test.go)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 8, 2026 09:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vmihailenco
vmihailenco merged commit 44c54f2 into uptrace:master Jul 17, 2026
4 checks passed
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.

3 participants