Skip to content

Conversation

@lzx404243
Copy link
Collaborator

Currently we are using yaml-cpp version 0.7.0. This PR bumps our yaml-cpp version to the just-released 0.8.0. Specifically, the following changes are made

  • Fixed the pre-commit hook to properly ignore yaml-cpp source files for formatting check
  • Updated lib/yamlcpp source to 0.8.0
  • Updated yaml version in the jsonrpc test and documentation

@lzx404243 lzx404243 self-assigned this Aug 18, 2023
@lzx404243 lzx404243 added the YAML label Aug 18, 2023
@lzx404243 lzx404243 added this to the 10.0.0 milestone Aug 18, 2023
@apache apache deleted a comment from bneradt Aug 18, 2023
@apache apache deleted a comment from bryancall Aug 18, 2023
@apache apache deleted a comment from ezelkow1 Aug 18, 2023
Copy link
Contributor

@bneradt bneradt left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks for working on this @lzx404243 . I'll let a non-Yahoo committer approve.

@bneradt bneradt merged commit 3a11dfe into apache:master Aug 20, 2023
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
* asf/master:
  Upgrade yaml-cpp version to 0.8.0 (apache#10249)
  This dependency is not needed, cmake did it right (apache#10250)
  Revert "Make OSX and FreeBSD not required temporarily (apache#10237)" (apache#10248)
  Python 3.12: microserver.test.ext wrap_socket update (apache#10247)
  Coverity 1508984: Dereference null return value (apache#10245)
  fall back to configure file for older cmake versions (apache#10236)
  TLS early data: logging updates (apache#10115)
  Fixing TSHttpTxnServerAddrSet (apache#10189)
  Make OSX and FreeBSD not required temporarily (apache#10237)
  Fixes a problem which can decrement milestone metrics unintentionally (apache#10188)
  Fix editor config for makefiles. (apache#10190)
  Correctly handle encoding for cache hash generation (apache#10126)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants