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
doc: removed dead links #3513
base: 1.15.0-dev
Are you sure you want to change the base?
doc: removed dead links #3513
Conversation
removes dead links from shared-libraries document and fixes some punctuation.
Although this is cool, I'm not sure if this documentation is correct. Looks like it was never ported, just someone doing Re: punctuation. Note that none of the list items are sentences, so normally those don't get punctuation? |
I don't mind at all.
The 'Parameters' list above the 'Script Flags' and 'Errors' lists uses punctuation, so I made the punctuation consistent across all lists. I can restore the punctuation back to how it was, if that's preferred. |
Let's see what in those lists is actually real documentation for this repo first. |
So, this document does not reflect implementation, at all. The replacement of So as of right now, I would not trust anything this document says. I see 2 options:
|
I would be fine with deprecating the whole library and documentation if it's not being used. Looks like bitcoin is doing the same bitcoin/bitcoin#29189. |
Bitcoin Core is replacing this library with another, more complete library, libbitcoinkernel. |
Do we have any plans to do the libbitcoinkernel thing to Dogecoin in the future? (seems like a lot of work) |
Personally, I think that having a well maintained shared library that exposes this codebase could help when testing alternative implementations and therefore would expect such a request from serious alternatives and auditors. But, having a non-maintained library like the one we have now (where for almost 5 years its docs could not have been more misleading, because every symbol is misnamed) is however a waste of everyone's time and effort. So unless someone is going to use it, I'd not prioritize that. |
I'm going to change this to draft and take it off the review board for now, because whatever we do, this change on its own doesn't make sense - i.e. will need to do work either way. However, since we already started the discussion here, we might as well have the discussion here, and give people some time to think about their wants/needs wrt this, and react. |
Sounds good, and I agree that it needs work either way. Just one note: if a solution isn't found before the next release, it would make me more comfortable if there was at least a label at the top of the document saying "this library is not actively maintained" or something to make it obvious that it's not useful at the moment. I just wouldn't want to see anyone waste their time or become confused by reading it. |
Documented a reminder in #3515 |
This removes 3 dead links from the
shared-libraries.md
document and adds some missing punctuation.