Skip to content

Conversation

wep21
Copy link
Contributor

@wep21 wep21 commented Oct 6, 2025

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @Vertexwahn, @vtsao-openai, modules you maintain (boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@wep21
Copy link
Contributor Author

wep21 commented Oct 6, 2025

@bazel-io skip_check unstable_url

@bazel-io bazel-io added the skip-url-stability-check Skip the URL stability check for the PR label Oct 6, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a new version 1.87.0.bcr.2 for the boost.test module, primarily to update its dependency on boost.thread. The review identified two critical issues that need to be addressed. First, there is a version mismatch in the MODULE.bazel file where the version should be 1.87.0.bcr.2. Second, the presubmit.yml file contains duplicate task names, which is invalid YAML and will cause presubmit jobs to be skipped.

@@ -0,0 +1,30 @@
module(
name = "boost.test",
version = "1.87.0.bcr.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

critical

The module version 1.87.0.bcr.1 does not match the directory version 1.87.0.bcr.2. Please update the version to match the directory path to ensure consistency and prevent potential resolution issues.

Suggested change
version = "1.87.0.bcr.1",
version = "1.87.0.bcr.2",

@wep21 wep21 force-pushed the update-boost-thread-deps branch from 336860b to bd422f9 Compare October 6, 2025 12:12
@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @vtsao-openai, @Vertexwahn, modules you maintain (boost.lockfree, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @vtsao-openai, modules you maintain (boost.lockfree, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @Vertexwahn, modules you maintain (boost.lockfree, boost.serialization, boost.spirit, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.serialization, boost.spirit, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @Vertexwahn, modules you maintain (boost.lockfree, boost.property_tree, boost.serialization, boost.spirit, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.property_tree, boost.serialization, boost.spirit, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @Vertexwahn, modules you maintain (boost.lockfree, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @lalten, modules you maintain (boost.log) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @Vertexwahn, modules you maintain (boost.lockfree, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.graph, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @Vertexwahn, modules you maintain (boost.graph, boost.lockfree, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.geometry, boost.graph, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 6, 2025

Hello @Vertexwahn, modules you maintain (boost.geometry, boost.graph, boost.lockfree, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@wep21
Copy link
Contributor Author

wep21 commented Oct 6, 2025

@fmeum @meteorcloudy I appreciate it if you could add presubmit auto run tag to run ci.

@fmeum fmeum added the presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval label Oct 6, 2025
vtsao-openai
vtsao-openai previously approved these changes Oct 6, 2025
bazel-io
bazel-io previously approved these changes Oct 6, 2025
Copy link
Member

@bazel-io bazel-io left a comment

Choose a reason for hiding this comment

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

All modules in this PR have been approved by their maintainers. This PR will be merged if all presubmit checks pass.

@wep21
Copy link
Contributor Author

wep21 commented Oct 6, 2025

error comes from boostorg/interprocess#258.
I will also add patch for interprocess.

@bazel-io bazel-io dismissed stale reviews from vtsao-openai and themself October 7, 2025 10:42

Require module maintainers' approval for newly pushed changes.

@bazel-io
Copy link
Member

bazel-io commented Oct 7, 2025

Hello @bazelbuild/bcr-maintainers, modules (boost.geometry, boost.graph, boost.interprocess, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@bazel-io
Copy link
Member

bazel-io commented Oct 7, 2025

Hello @Vertexwahn, modules you maintain (boost.geometry, boost.graph, boost.interprocess, boost.lockfree, boost.log, boost.property_tree, boost.serialization, boost.spirit, boost.test, boost.xpressive) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@wep21 wep21 force-pushed the update-boost-thread-deps branch from 7a447d7 to e996f23 Compare October 7, 2025 12:17
@wep21
Copy link
Contributor Author

wep21 commented Oct 7, 2025

@fmeum @meteorcloudy ready to review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval skip-url-stability-check Skip the URL stability check for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants