-
Notifications
You must be signed in to change notification settings - Fork 683
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
Simple docs update request #5879
Comments
@obycode i think the question here is if @0xpessimist is correct in that the text is missing from the source code (and downstream in the documentation), and if it needs to be added. if true, we can put together a PR to address. |
That is correct, but is not a corner case, so I don't think it needs to be added. Actually, I might remove one of the existing bullet points, "if |
By "corner case", what we mean is something where the code may operate differently from what you would expect mathematically. The case when |
Yes, I felt the need to add this too because "if |
since you found this @0xpessimist , could you put together the pr with brice's suggestion? we can retcon the docs until the change is released as well |
I opened a PR on
stacks-network/docs
to update the documentation (here: stacks-network/docs#1710), I was asked to open an issue here because it was auto-generated. That documentation for thepow
function appears to come from:stacks-core/clarity/src/vm/docs/mod.rs
Lines 493 to 508 in 2caa9bf
And I want to add;
i2
is0
, return1
The text was updated successfully, but these errors were encountered: