Skip to content
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

update rules_boost to latest version #1898

Merged
merged 2 commits into from
Sep 6, 2022

Conversation

Wyverald
Copy link
Contributor

Want to contribute? Great! Make sure you've read and understood
CONTRIBUTING.md.

@Wyverald
Copy link
Contributor Author

gentle ping. Cartographer remains broken in the downstream pipeline, and if we can't resolve that soon, we'd need to disable Cartographer on Bazel CI.

Copy link
Member

@wohe wohe left a comment

Choose a reason for hiding this comment

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

Other than the missing SHA256 this looks good to me. Once you add it, I'd be happy to merge.

bazel/repositories.bzl Show resolved Hide resolved
Signed-off-by: Xùdōng Yáng <[email protected]>
@wohe wohe merged commit ef00de2 into cartographer-project:master Sep 6, 2022
kscottz pushed a commit that referenced this pull request Dec 8, 2023
Signed-off-by: Xùdōng Yáng <[email protected]>
Signed-off-by: kscottz <[email protected]>
kscottz added a commit that referenced this pull request Jan 5, 2024
* update rules_boost to latest version (#1898)

Signed-off-by: Xùdōng Yáng <[email protected]>
Signed-off-by: kscottz <[email protected]>

* Add note for ROS users.

Signed-off-by: kscottz <[email protected]>

* spelling errors.

Signed-off-by: kscottz <[email protected]>

---------

Signed-off-by: Xùdōng Yáng <[email protected]>
Signed-off-by: kscottz <[email protected]>
Co-authored-by: Xùdōng Yáng <[email protected]>
@Wyverald Wyverald deleted the patch-1 branch July 23, 2024 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants