Retire fixTakerDryOfferRemoval amendment - #5958
Conversation
Signed-off-by: Pratik Mankawde <pmankawde@ripple.com>
bthomee
left a comment
There was a problem hiding this comment.
It looks like this fix amendment had implicitly already been permanently enabled due to other changes in the past.
|
@pratikmankawde I don't see why, but the OfferAllFeatures test is failing consistently. Can you please take a look? On Linux I see |
Signed-off-by: Pratik Mankawde <pmankawde@ripple.com>
I made the fixes and tested locally. Seems to be working. Waiting for the tests to pass on CI. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #5958 +/- ##
=========================================
- Coverage 78.3% 78.3% -0.0%
=========================================
Files 817 817
Lines 68944 68944
Branches 8304 8307 +3
=========================================
- Hits 53989 53985 -4
- Misses 14955 14959 +4 🚀 New features to boost your workflow:
|
|
The |
Amendments activated for more than 2 years can be retired. This change retires the fixTakerDryOfferRemoval amendment.
High Level Overview of Change
This change retires the
fixTakerDryOfferRemovalamendment.Context of Change
Amendments activated for more than 2 years can be retired.
https://xrpl.org/resources/known-amendments#fixTakerDryOfferRemoval
Type of Change
.gitignore, formatting, dropping support for older tooling)