refactor: rename testing pacakge to rotesting#233
Conversation
|
Claude finished @samber's task in 1m 2s —— View job PR Review: Rename testing package to rotestingSummaryThis PR correctly renames the internal Changes Verified
Breaking Change NoticeThis is a breaking change for downstream users who currently import import rotesting "github.com/samber/ro/testing"RecommendationLGTM with the note that this should be called out in release notes as a breaking change. |
There was a problem hiding this comment.
Pull request overview
Renames the internal Go package name for the /testing import path to rotesting (to reduce collisions with the stdlib testing package) and updates usages in the testify plugin and documentation accordingly.
Changes:
- Rename package declaration from
package testingtopackage rotestingin the/testingdirectory. - Update
plugins/testifyto import and referencerotesting.AssertSpec. - Update docs to show importing the
/testingpackage asrotesting(but one snippet currently has a typo).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| testing/spec.go | Renames package declaration to rotesting. |
| testing/assert.go | Renames package declaration to rotesting. |
| testing/assert_test.go | Renames package declaration to rotesting. |
| plugins/testify/testify.go | Updates import alias and type references from testing to rotesting. |
| docs/docs/testing.md | Updates documentation imports/usages to rotesting (one usage is currently incorrect). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #233 +/- ##
=======================================
Coverage 65.04% 65.04%
=======================================
Files 95 95
Lines 9104 9104
=======================================
Hits 5922 5922
Misses 3088 3088
Partials 94 94
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.