-
Notifications
You must be signed in to change notification settings - Fork 677
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
Update docs to reflect segwit support in Stacks 2.1 #5480
base: develop
Are you sure you want to change the base?
Update docs to reflect segwit support in Stacks 2.1 #5480
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have fix it, now the sentence syntax is correct in English. Thanks
Thanks for reaching out. I have fixed it so that sentence syntax is correct
in English
…On Tue, Nov 19, 2024, 2:25 PM Adriano Di Luzio ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In stackslib/src/chainstate/stacks/boot/docs.rs
<#5480 (comment)>
:
> @@ -33,7 +33,7 @@ This is the self-service interface. tx-sender will be the Stacker.
* You will need the minimum uSTX threshold. This isn't determined until the reward cycle begins, but this
method still requires stacking over the _absolute minimum_ amount, which can be obtained by calling `get-stacking-minimum`.
* The pox-addr argument must represent a valid reward address. Right now, this must be a Bitcoin
-p2pkh or p2sh address. It cannot be a native Segwit address, but it may be a p2wpkh-p2sh or p2wsh-p2sh address.
+p2pkh or p2sh address. // Native Segwit addresses are now supported as of Stacks 2.1. but it may be a p2wpkh-p2sh or p2wsh-p2sh address.
Thank you! Could you fix this so that the sentence syntax is correct in
English?
—
Reply to this email directly, view it on GitHub
<#5480 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BLBHANZDFF5PI6UHOWQVEDD2BM34XAVCNFSM6AAAAABSCBFIRSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINBVGQ3TGNZZHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
@jcnelson I hope you're doing well. I wanted to provide some context regarding my pull request (#5480) and seek your guidance on resolving the mutation testing issue. In my contribution, I only made a small change to update the message that previously stated "Not a native segwit address" to reflect that "Native segwit address is now supported as of Stacks 2.1." I didn't touch any other part of the codebase. However, I received a failure notification regarding mutation testing, and my status now shows "Action Required." Since I don't have knowledge of how to address this issue, I would greatly appreciate your help in identifying what went wrong and what steps I can take to fix it. Thank you for your time and support! Please let me know if you need any further details about my changes. |
…0.0.0.4 merge Release/3.0.0.0.4 to master
The base branch was changed.
Description: This pull request updates the documentation regarding Segwit address support in the Stacks project. The current documentation stated that "It cannot be a native Segwit address," which was accurate before Stacks 2.1. However, with the release of Stacks 2.1, native Segwit addresses are now supported, and this change has been reflected in the documentation.
Applicable issues
N/A
The statement "It cannot be a native Segwit address" was incorrect after Stacks 2.1. It is now corrected to reflect the current capabilities.
Additional info (benefits, drawbacks, caveats)
This change ensures the documentation accurately reflects the capabilities of Stacks 2.1, providing clarity for developers using Segwit addresses.
Checklist
docs/rpc/openapi.yaml
andrpc-endpoints.md
for v2 endpoints,event-dispatcher.md
for new events)clarity-benchmarking
repobitcoin-tests.yml