-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move ./pf, ./testing and ./x/muxer under ./pkg #2477
Conversation
This change is part of the following stack: Change managed by git-spice. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2477 +/- ##
==========================================
- Coverage 61.77% 61.76% -0.01%
==========================================
Files 380 380
Lines 51263 51263
==========================================
- Hits 31667 31665 -2
- Misses 17780 17783 +3
+ Partials 1816 1815 -1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
This removes three unnecessary modules: * pf/go.mod * x/muxer/go.mod * testing/go.mod And three unnecessary test modules: * pf/tests/go.mod * pkg/tests/go.mod * x/muxer/tests/go.mod This is intended to be merged with #2477 and #2480, which combine to make this a no-op for existing consumers. Refs #1445.
This PR has been shipped in release v3.93.0. |
This PR has been shipped in release v3.93.1. |
This moves
pf
,testing
andx/muxer
underpkg
. This will allow us torestore modules at those paths in order to not break existing consumers.
More specifically, the new import paths are:
github.com/pulumi-terraform-bridge/pf
→github.com/pulumi-terraform-bridge/v3/pkg/pf
github.com/pulumi-terraform-bridge/x/muxer
→github.com/pulumi-terraform-bridge/v3/pkg/x/muxer
github.com/pulumi-terraform-bridge/testing
→github.com/pulumi-terraform-bridge/v3/pkg/x/testing
(I put this underx
sincetesting/x
is the only package in this module)