Skip to content

Conversation

a1q123456
Copy link
Collaborator

High Level Overview of Change

As discussed, if a feature is never voted on then it should be safe to remove.

@a1q123456 a1q123456 marked this pull request as ready for review July 10, 2025 17:36
@a1q123456 a1q123456 requested a review from a team as a code owner July 10, 2025 17:36
Copy link

codecov bot commented Jul 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.4%. Comparing base (c233df7) to head (64f5851).
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           develop   #5550   +/-   ##
=======================================
  Coverage     78.4%   78.4%           
=======================================
  Files          816     816           
  Lines        71733   71730    -3     
  Branches      8598    8598           
=======================================
- Hits         56238   56237    -1     
+ Misses       15495   15493    -2     
Files with missing lines Coverage Δ
include/xrpl/protocol/Feature.h 100.0% <ø> (ø)
src/libxrpl/protocol/Feature.cpp 94.8% <100.0%> (-0.1%) ⬇️

... and 4 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.

Signed-off-by: JCW <[email protected]>
…ee-and-XRPL_ABANDON' into a1q123456/rip-out-owner-pays-fee-and-XRPL_ABANDON
Copy link
Collaborator

@ximinez ximinez 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!

@a1q123456 a1q123456 requested a review from Bronek July 22, 2025 16:23
@a1q123456 a1q123456 added the Needs second review PR requires at least one more code review approval before it can be merged label Jul 22, 2025
@bthomee bthomee removed the Needs second review PR requires at least one more code review approval before it can be merged label Jul 23, 2025
@a1q123456 a1q123456 added Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. Added to API Changelog API changes have been documented in API-CHANGELOG.md and removed Added to API Changelog API changes have been documented in API-CHANGELOG.md labels Jul 23, 2025
@bthomee bthomee enabled auto-merge (squash) July 23, 2025 13:01
@bthomee bthomee merged commit faa781b into develop Jul 23, 2025
25 checks passed
@bthomee bthomee deleted the a1q123456/rip-out-owner-pays-fee-and-XRPL_ABANDON branch July 23, 2025 13:27
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants