Skip to content

Conversation

jslee02
Copy link
Member

@jslee02 jslee02 commented May 18, 2025

Remove this line and describe this pull request. Link to relevant GitHub issues, if any.


Before creating a pull request

  • Document new methods and classes
  • Format new code files using ClangFormat by running make format
  • Build with -DDART_TREAT_WARNINGS_AS_ERRORS=ON and resolve all the compile warnings

Before merging a pull request

  • Set version target by selecting a milestone on the right side
  • Summarize this change in CHANGELOG.md
  • Add unit test(s) for this change
  • Add Python bindings for new methods and classes

@jslee02 jslee02 marked this pull request as ready for review May 18, 2025 00:57
@jslee02 jslee02 requested a review from Copilot May 18, 2025 00:57
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates various build and testing dependencies to newer versions to keep the project up to date and benefit from bug fixes and features in the latest releases.

  • Bumps build-dependencies versions (benchmark, black, cmake, gtest, isort, pip, pytest, setuptools)
  • Maintains < upper bounds to avoid major unexpected upgrades
  • Ensures tooling aligns with current project requirements
Comments suppressed due to low confidence (2)

pixi.toml:13

  • Upgrading to CMake 4.x can introduce breaking changes; please verify compatibility with CMake 4 or constrain the version if support hasn't been confirmed.
cmake = ">=4.0.2,<5"

pixi.toml:11

  • After bumping Black to >=25.1.0, re-run the code formatter to apply any new style rules and ensure the codebase remains consistent.
black = ">=25.1.0,<26"

@jslee02 jslee02 merged commit 49e851b into main May 18, 2025
2 of 25 checks passed
@jslee02 jslee02 deleted the upgrade_pkgs_in_pixi branch May 18, 2025 01:02
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.

1 participant