Skip to content

Trying to fix clang-format#5598

Merged
bthomee merged 2 commits intodevelopfrom
Bronek/fixing_clang_format
Jul 23, 2025
Merged

Trying to fix clang-format#5598
bthomee merged 2 commits intodevelopfrom
Bronek/fixing_clang_format

Conversation

@Bronek
Copy link
Copy Markdown
Collaborator

@Bronek Bronek commented Jul 23, 2025

High Level Overview of Change

Trying to fix clang-format workflow

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Performance (increase or change in throughput and/or latency)
  • Tests (you added tests for code that already exists, or your new feature included in this PR)
  • Documentation update
  • Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
  • Release

@Bronek Bronek force-pushed the Bronek/fixing_clang_format branch from 55169ab to ecab3c2 Compare July 23, 2025 10:00
@Bronek Bronek added the Trivial Simple change with minimal effect, or already tested. Only needs one approval. label Jul 23, 2025
@Bronek Bronek requested review from a1q123456 and bthomee July 23, 2025 10:03
@Bronek Bronek added the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label Jul 23, 2025
@bthomee bthomee enabled auto-merge (squash) July 23, 2025 10:16
Copy link
Copy Markdown
Contributor

@a1q123456 a1q123456 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.4%. Comparing base (6090965) to head (ab0441a).
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           develop   #5598   +/-   ##
=======================================
  Coverage     78.4%   78.4%           
=======================================
  Files          816     816           
  Lines        71714   71714           
  Branches      8598    8578   -20     
=======================================
+ Hits         56226   56248   +22     
+ Misses       15488   15466   -22     

see 9 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bthomee bthomee merged commit 7ff4f79 into develop Jul 23, 2025
3 checks passed
@bthomee bthomee deleted the Bronek/fixing_clang_format branch July 23, 2025 10:44
mvadari added a commit that referenced this pull request Jul 23, 2025
Co-authored-by: Jingchen <a1q123456@users.noreply.github.com>
Co-authored-by: Bronek Kozicki <brok@incorrekt.com>
Co-authored-by: Ayaz Salikhov <mathbunnyru@users.noreply.github.com>
Co-authored-by: Denis Angell <dangell@transia.co>
Co-authored-by: Vlad <129996061+vvysokikh1@users.noreply.github.com>
Co-authored-by: Shawn Xie <35279399+shawnxie999@users.noreply.github.com>
Co-authored-by: yinyiqian1 <yqian@ripple.com>
Co-authored-by: Michael Legleux <legleux@users.noreply.github.com>
Co-authored-by: Bart <bthomee@users.noreply.github.com>
Co-authored-by: Chenna Keshava B S <21219765+ckeshava@users.noreply.github.com>
Co-authored-by: Vito Tumas <5780819+Tapanito@users.noreply.github.com>
Co-authored-by: Luc des Trois Maisons <3maisons@gmail.com>
Co-authored-by: Valentin Balaschenko <13349202+vlntb@users.noreply.github.com>
fix: crash when trace-logging in tests (#5529)
Fix compilation error with clang-20 and cleanup (#5543)
fix: Link with boost libraries explicitly (#5546)
fix: add allowTrustLineLocking flag for account_info (#5525)
fixAMMClawbackRounding: adjust last holder's LPToken balance (#5513)
Fix macos runner (#5585)
Fix clang-format CI job (#5598)
resolved actions/runner#2058.
ximinez added a commit that referenced this pull request Jul 24, 2025
…to ximinez/lending-refactoring-3

* XRPLF/ximinez/lending-refactoring-2:
  refactor: Update rocksdb (#5568)
  Switch instrumentation workflow to use dependencies (#5607)
  chore: Rename conan profile to `default` (#5599)
  Include `network_id` in validations and subscription stream responses (#5579)
  Add support for `DomainID` in `MPTokenIssuance` transactions (#5509)
  chore: Remove unused code after flow cross retirement (#5575)
  Remove obsolete owner pays fee feature and XRPL_ABANDON stanza (#5550)
  refactor: Makes HashRouter flags more type-safe (#5371)
  Fix clang-format CI job (#5598)
This was referenced Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. Trivial Simple change with minimal effect, or already tested. Only needs one approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants