update LICENSE info#262
Conversation
thepastaclaw
left a comment
There was a problem hiding this comment.
Code Review
Documentation-only PR that reorders the README copyright lines and removes the Bitcore trademark sentence. Verified against the diff: the previous lines ended with two trailing spaces (forcing Markdown hard breaks); the new lines do not, so the three copyright notices will render as a single run-on paragraph on GitHub/npm. LICENSE file is untouched and remains out of sync, but that predates the PR.
💬 1 nitpick(s)
Out-of-scope follow-up suggestions (1)
These are valid observations, but they are outside this PR's scope and should be handled in separate issues or author/maintainer-requested PRs rather than blocking this review.
- LICENSE copyright years and entities are out of sync with README — The checked-in
LICENSEstill readsCopyright (c) 2017-2018 Dash Core Group, Inc.and does not list The Dash Foundation, Inc., while README now reads2017-2020 Dash Core Group, Inc.and lists The Dash Foundation. The divergence predates this PR (LICENSE is not modified here despite the commit message), and the PR's stated scope is the README attribution footer only, so this is not a blocker — but the two files should be reconciled separately.- Follow-up: Open a separate maintainer-requested PR to reconcile
LICENSEcopyright years and listed entities with the README attribution (or vice versa).
- Follow-up: Open a separate maintainer-requested PR to reconcile
| Copyright 2017-2020 Dash Core Group, Inc. | ||
| Copyright 2016-2017 The Dash Foundation, Inc. | ||
| Copyright 2013-2017 BitPay, Inc. |
There was a problem hiding this comment.
💬 Nitpick: Copyright lines lose hard line breaks and render as one run-on paragraph
Verified against the diff: the removed lines ended with two trailing spaces (GFM hard-break markers), and the replacement lines do not. In GitHub-Flavored Markdown, consecutive non-blank lines inside the same paragraph are joined with a single space on render unless each non-final line ends with two trailing spaces, a backslash, or <br>. As-is, the three copyright notices will display as one wrapped paragraph (Copyright 2017-2020 Dash Core Group, Inc. Copyright 2016-2017 The Dash Foundation, Inc. Copyright 2013-2017 BitPay, Inc.) instead of three separate lines as before. Restoring the two trailing spaces on the first two lines preserves the prior rendering.
| Copyright 2017-2020 Dash Core Group, Inc. | |
| Copyright 2016-2017 The Dash Foundation, Inc. | |
| Copyright 2013-2017 BitPay, Inc. | |
| Copyright 2017-2020 Dash Core Group, Inc. | |
| Copyright 2016-2017 The Dash Foundation, Inc. | |
| Copyright 2013-2017 BitPay, Inc. |
source: ['claude']
Dash is the current maintainer of this software, Dash comes first.
We don't use the Bitpay / Bitcore trademarks, we don't need to mention them or draw additional attention to them.