-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix small typos #5279
Fix small typos #5279
Conversation
@code0xff Can you affirm this is ready to merge? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #5279 +/- ##
=========================================
- Coverage 78.2% 78.2% -0.0%
=========================================
Files 790 790
Lines 67639 67638 -1
Branches 8160 8164 +4
=========================================
- Hits 52869 52862 -7
- Misses 14770 14776 +6
|
Yes, this is ready to merge. The changes are just simple typo fixes. Please let me know if there's anything else that needs to be done. |
@code0xff would you be able to sign your commits?
|
High Level Overview of Change
Context of Change
Type of Change
.gitignore
, formatting, dropping support for older tooling)API Impact
libxrpl
change (any change that may affectlibxrpl
or dependents oflibxrpl
)