chore(azurite): tidy module dependencies - #3838
Conversation
Re-tidy modules/azurite to pick up the azblob v1.8.0 checksum bump from the recent Azure SDK dependency updates in modules/azure.
✅ Deploy Preview for testcontainers-go ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
What does this PR do?
Runs
make tidy-alland commits the only resulting diff:modules/azurite/go.sumnow recordsgithub.com/Azure/azure-sdk-for-go/sdk/storage/azblobatv1.8.0instead ofv1.6.0(both the module andgo.modchecksum lines).modules/azurite/go.modis unchanged — azblob is a transitive dependency pulled in through the localreplaceonmodules/azure, so only the lockfile was stale. No other module in the repo produced a diff.Why is it important?
#3805 bumped azblob to
v1.8.0inmodules/azure, butmodules/azuriteconsumesmodules/azurevia areplacedirective, so itsgo.sumwas left pinning the oldv1.6.0checksums. That leaves the repo in a state wherego mod tidyis not a no-op forazurite, which makes the tidy check noisy for unrelated PRs touching that module.Related issues
How to test this PR
go build,go vetandgo test ./...all pass locally formodules/azurite(tests run against a real Azurite container, ~9s).