-
Notifications
You must be signed in to change notification settings - Fork 8
auto-docs: Update Go modules #294
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
Conversation
✅ Deploy Preview for docs-ui ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 📝 WalkthroughWalkthroughThe pull request updates the version of the dependency Possibly related PRs
Suggested reviewers
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
blobl-editor/wasm/go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
blobl-editor/wasm/go.mod(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
blobl-editor/wasm/go.mod
Outdated
| require ( | ||
| github.com/redpanda-data/benthos/v4 v4.51.0 | ||
| github.com/redpanda-data/connect/v4 v4.55.0 | ||
| github.com/redpanda-data/connect/v4 v4.55.1 |
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.
💡 Verification agent
🧩 Analysis chain
Verify patch-level module update for backward compatibility
Bumping github.com/redpanda-data/connect/v4 from v4.55.0 to v4.55.1 is a patch release—this should be safe, but please ensure there are no regressions:
🏁 Script executed:
#!/bin/bash
# Ensure module consistency and run the full test suite
cd blobl-editor/wasm
go mod tidy
go test ./...Length of output: 7371
Fix failing wasm tests by running them in the JS/WASM environment
The go test invocation is failing because the package imports syscall/js, which only builds under GOOS=js and GOARCH=wasm. Before verifying the patch-level bump, update your test command or CI configuration:
• In blobl-editor/wasm, run tests with the JS/WASM environment:
cd blobl-editor/wasm
export GOOS=js GOARCH=wasm
go test ./...• (Alternatively) add the js build tag:
cd blobl-editor/wasm
go test -tags=js ./...Once the tests pass under the correct environment, confirm that bumping github.com/redpanda-data/connect/v4 from v4.55.0 to v4.55.1 introduces no regressions.
🤖 Prompt for AI Agents
In blobl-editor/wasm/go.mod at line 9, the module
github.com/redpanda-data/connect/v4 was updated from v4.55.0 to v4.55.1, a patch
release. To verify this update does not cause regressions, run the tests in the
correct JS/WASM environment since the package imports syscall/js. Fix the
failing tests by running the test command with GOOS=js and GOARCH=wasm
environment variables set or by adding the js build tag. After tests pass
successfully, confirm the patch-level bump is safe.
3654800 to
b6f7623
Compare
b6f7623 to
45fdf26
Compare
This PR updates the Go modules for the Bloblang playground Wasm code.