Skip to content
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

Updated URL of the JavaScript reference implementation #1127

Merged
merged 1 commit into from
Jul 2, 2021

Conversation

chirag64
Copy link
Contributor

@chirag64 chirag64 commented Jun 6, 2021

Old repo is archived and links to the new repo are found in the old one's README file. This change updates the old repo's link to the new one.

Old repo is archived and links to the new repo are found in the old one's README file. This change updates the old repo's link to the new one.
@michaelfolkson
Copy link
Contributor

The old repo is archived but if you visit that old repo it is clear that it has been archived and that it has moved. In addition to this BIP being "Unanimously Discourage for implementation" I personally don't think it is worth asking the BIP authors to ACK this. But if one of them does ACK it it can be merged as it is better if it links to the new repo.

@luke-jr
Copy link
Member

luke-jr commented Jul 2, 2021

@prusnak
Copy link
Contributor

prusnak commented Jul 2, 2021

ACK

@luke-jr luke-jr merged commit ad745f2 into bitcoin:master Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants